Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Andromatic to Support Preinstalled SDK Tools #10

Open
techware01 opened this issue Sep 19, 2024 · 1 comment
Open

Enhance Andromatic to Support Preinstalled SDK Tools #10

techware01 opened this issue Sep 19, 2024 · 1 comment

Comments

@techware01
Copy link

techware01 commented Sep 19, 2024

Hi,

I would like to suggest a feature that would allow Andromatic to utilize preinstalled SDK tools such as sdkmanager, platform-tools, and cmdline-tools.

To achieve this, Andromatic could first check for the required tools in the $ANDROID_HOME directory before proceeding with any installations. Additionally, it could accept the installation path as an argument, with automatic installation serving as a fallback option.

For instance @wtto00/android-tools uses the first two options.

It would be beneficial from a security perspective to have the option to control the installation process manually, rather than allowing Andromatic to automatically download and install binaries (including Java) in the background.

While I like that the current approach simplifies the installation process by eliminating the complexities of manual setup, it would be great to provide users with the choice of how the tools should be installed.

@baltpeter
Copy link
Member

Not using the $ANDROID_HOME was a deliberate decision. I feel like automatically changing/installing packages in $ANDROID_HOME without prompting the user would be quite "rude", but that's exactly what andromatic is supposed to do in the context of the Tweasel tools.

I could see andromatic being more useful for other use cases, though, if there was an option to alternatively use an existing directory, as you also propose. I think implementing that would be a good additional feature.

However, I'm not quite sure how that would look given the current API. Since all of the functions stand on their own, we don't really have anywhere we could add this option…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants