-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Update ANDROID_HOME to ANDROID_SDK_ROOT #2590
Conversation
From https://developer.android.com/studio/command-line/variables: > ANDROID_HOME, which also points to the SDK installation directory, is deprecated.
Deploy preview for react-native ready! Built with commit 68ffc28 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @sonicdoe, thank you for reporting this out and improving asset, however it might be a bit early call, since as stated in the linked docs:
If
ANDROID_HOME
is defined and contains a valid SDK installation, its value is used instead of the value inANDROID_SDK_ROOT
.
Which might create other problems or confusion for the users who migrated to the new name. I will request review from others, so we can figure out if it's a good time to update this.
Please also update the Android guide for macOS:
I was surprised about that rule as well. It makes it sound like
It’s already updated, see this diff. |
Hello @sonicdoe, sorry, but I have lost a track of this PR for some reason. 😅 It look like the path changes have already been applied within #2991, so I think we can close this PR now. If you seeing anything incorrect or missing in the parts after the refactor please post the new PR with the suggestions! |
thanks too @sonicdoe btw i'm seeing Last updated 2022-03-23 UTC so i think Google have made a u-turn 20 days after @Simek i'm happy to create a new issue and PR top revert if i'm seeing straight 😅 (i'm seeing both variables remain on latest GitHub Actions Virtual Environments, so as mentioned both variables will be checked by AGP for consistency anway) |
Yup you're right. We should update the docs to use |
From the Android Studio user guide: