-
Notifications
You must be signed in to change notification settings - Fork 18
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 SDK (33.0.1 - Android 13 - API 33) #256
Conversation
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.
lgtm. Perhaps add a small version comment to the Dockerfile
server/docker-base/Dockerfile
Outdated
ENV PATH ${PATH}:${ANDROID_NDK20_BIN_PATH} | ||
|
||
# | ||
# New NDK25b: |
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.
It's usually good to put the version of Defold where it's effectively in (so in this case 1.4.2)
That way it's easier for us to remove old stuff after 6 months.
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.
I'm going to merge it into the next version (after 1.4.2)
so, I'll add it before merge when I know the next version number
server/docker-base/Dockerfile
Outdated
@@ -284,6 +340,7 @@ RUN \ | |||
chmod +r -R ${ANDROID_ROOT} && \ | |||
chmod +w -R ${ANDROID_SDK_ROOT} && \ | |||
chmod -R 755 ${ANDROID_ROOT}/android-ndk-r${ANDROID_NDK20_VERSION} && \ | |||
chmod -R 755 ${ANDROID_ROOT}/android-ndk-r${ANDROID_NDK25_VERSION} && \ | |||
# check that dx installed properly |
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.
Perhaps want something similar for ndk25 ?
#cleanup | ||
rm -rf ${ANDROID_HOME}/extras && \ |
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.
Save about %50 size of the SDK
@JCash now PR has only Android SDK update, without NDK fixes. You can take a look this more clean PR |
No description provided.