Skip to content
This repository was archived by the owner on Mar 31, 2022. It is now read-only.

Deprecated SDK Tools replaced with Command line Tools#300

Merged
lmesz-bitrise merged 10 commits intobitrise-io:masterfrom
DroidsOnRoids:commandline-tools-2.0
Jul 24, 2020
Merged

Deprecated SDK Tools replaced with Command line Tools#300
lmesz-bitrise merged 10 commits intobitrise-io:masterfrom
DroidsOnRoids:commandline-tools-2.0

Conversation

@koral--
Copy link
Copy Markdown
Contributor

@koral-- koral-- commented Jun 14, 2020

Pull Request Checklist

Check/accept these:

  • The tool I added is stable, and does not require frequent updates. Preinstalled tools on the LTS stacks
    are not updated, so if the tool requires frequent updates you should handle the installation on-demand (see: Install Any Additional Tool - DevCenter for more information)
  • I've updated the version in the Dockerfile
  • I've updated the CHANGELOG.md
  • I've provided a link or explanation below which describes the changes in the tool itself
  • I added a version report line to system_report.sh for the new tool(s) I added

https://developer.android.com/studio/releases/sdk-tools

@koral-- koral-- force-pushed the commandline-tools-2.0 branch from 947a3ef to ecbdf7c Compare June 14, 2020 12:38
Szabadember
Szabadember previously approved these changes Jun 15, 2020
Comment thread Dockerfile Outdated
Comment thread Dockerfile
&& unzip -q android-sdk-tools.zip -d ${ANDROID_HOME} \
&& rm android-sdk-tools.zip
&& wget -q https://dl.google.com/android/repository/commandlinetools-linux-6514223_latest.zip -O android-commandline-tools.zip \
&& mkdir -p ${ANDROID_HOME}/cmdline-tools \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
&& mkdir -p ${ANDROID_HOME}/cmdline-tools \
&& mkdir -p ${ANDROID_HOME} \

Actually there is no need for this folder, and that way the whole PATH change is unnecessary as well. We should stick to the $ANDROID_HOME/tools/bin structure

Copy link
Copy Markdown
Contributor Author

@koral-- koral-- Jun 16, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lmesz-bitrise
lmesz-bitrise previously approved these changes Jun 16, 2020
@koral--
Copy link
Copy Markdown
Contributor Author

koral-- commented Jun 18, 2020

@lmesz-bitrise @Szabadember @fehersanyi-bitrise @ghaabor could you paste relevant build log part?

@lmesz-bitrise
Copy link
Copy Markdown
Contributor

It could be some intermittent hiccup cause:
Step 1/35 : FROM quay.io/bitriseio/bitrise-base:alpha received unexpected HTTP status: 502 Bad Gateway

@koral--
Copy link
Copy Markdown
Contributor Author

koral-- commented Jun 25, 2020

@lmesz-bitrise @fehersanyi-bitrise is there any update on this?

@lmesz-bitrise
Copy link
Copy Markdown
Contributor

Sorry, I just restarted your build but looks like the problem is around the change.

Removing intermediate container 311b573b6459 ---> 102773b2bfd6 Step 7/35 : RUN cd /opt && wget -q https://dl.google.com/android/repository/commandlinetools-linux-6514223_latest.zip -O android-commandline-tools.zip && mkdir -p ${ANDROID_HOME}/cmdline-tools/latest && unzip -q android-commandline-tools.zip -d ${ANDROID_HOME}/cmdline-tools/latest && rm android-commandline-tools.zip ---> Running in fc308a550cbb Removing intermediate container fc308a550cbb ---> d07ead7a30e6 Step 8/35 : ENV PATH ${PATH}:${ANDROID_HOME}/platform-tools:${ANDROID_HOME}/cmdline-tools/latest/tools/bin ---> Running in dd454584a577 Removing intermediate container dd454584a577 ---> 482362ab3643 Step 9/35 : RUN yes | sdkmanager --licenses ---> Running in 0d6621222b0e Warning: Could not create settings java.lang.IllegalArgumentException at com.android.sdklib.tool.sdkmanager.SdkManagerCliSettings.<init>(SdkManagerCliSettings.java:428) at com.android.sdklib.tool.sdkmanager.SdkManagerCliSettings.createSettings(SdkManagerCliSettings.java:152) at com.android.sdklib.tool.sdkmanager.SdkManagerCliSettings.createSettings(SdkManagerCliSettings.java:134) at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:57) at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48) Usage: sdkmanager [--uninstall] [<common args>] [--package_file=<file>] [<packages>...] sdkmanager --update [<common args>] sdkmanager --list [<common args>] sdkmanager --licenses [<common args>] sdkmanager --version

@bitrisekristof
Copy link
Copy Markdown
Contributor

@koral-- Sorry for the late reply. We are OK with this change but the CI is red and you have to update. :)

@koral--
Copy link
Copy Markdown
Contributor Author

koral-- commented Jul 9, 2020

OK, thanks for the info.

I've first tried to keep tools component updating added by @trapacska in e2b0e6e. Now its equivalent is called commandline-tools;latest.
We can install commandline-tools;latest (or any other fixed version) using sdkmanager but it won't be used out of the box because it is installed to a different path (it also contain /latest/) which is not in $PATH. The latter path cannot be used since the beginning due to this issue: https://stackoverflow.com/questions/60440509/android-command-line-tools-sdkmanager-always-shows-warning-could-not-create-se.
What is more it seems that tools update was never effective because the latest release was in September 2017 while linked PR was merged in Novemeber.

So taking that all into account I've just removed SDK tools/Command line tools updating to the latest version at the time of building Docker image.

@koral--
Copy link
Copy Markdown
Contributor Author

koral-- commented Jul 23, 2020

@lmesz-bitrise @fehersanyi-bitrise any update on this?

@lmesz-bitrise lmesz-bitrise merged commit 5866d6d into bitrise-io:master Jul 24, 2020
@lmesz-bitrise
Copy link
Copy Markdown
Contributor

Thank you for the PR and sorry for the delay

@koral-- koral-- deleted the commandline-tools-2.0 branch July 24, 2020 12:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants