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

Update android gradle plugins versions #43178

Merged
merged 2 commits into from
Oct 29, 2020

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Oct 29, 2020

Bump the Android gradle plugins versions to the latest.

Android Studio 4.1+ seems to stop providing editor support for (and recognizing) Android libraries when they define an external native build whose root includes the library's directory.
As a workaround, I moved the external native build definition into a separate empty Android library. This does not affect the project structure since the external native build definition was added in the first place for Android Studio editor support (the actual native dependencies are handled by scons).

@m4gr3d m4gr3d added platform:android topic:buildsystem cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Oct 29, 2020
@m4gr3d m4gr3d added this to the 4.0 milestone Oct 29, 2020
@m4gr3d m4gr3d requested review from pouleyKetchoupp, akien-mga and a team October 29, 2020 07:13
@@ -6,7 +6,8 @@ ext.versions = [
buildTools : '30.0.1',
supportCoreUtils : '1.0.0',
kotlinVersion : '1.4.10',
v4Support : '1.0.0'
v4Support : '1.0.0',
javaVersion : 1.8
Copy link
Member

Choose a reason for hiding this comment

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

Do you know what's the state of Java 11 compatibility in the Android SDK / Studio?

Our documentation still tells people to install specifically JDK 8 for jarsigner, as earlier and later versions seemed to be problematic, and I remember running into issues with newer Java when using sdkmanager.

But many Linux distros are finally leaving OpenJDK 1.8 behind and defaulting to JDK 11 LTS, so we'll need to support this somehow if we can.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@akien-mga I'll look into it.

compileSdk : 29,
minSdk : 18,
targetSdk : 29,
buildTools : '29.0.3',
buildTools : '30.0.1',
Copy link
Member

Choose a reason for hiding this comment

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

I'm not super knowledgeable on how buildTools relate to the target SDK, but do we want to bump compileSdk and targetSdk to 30 eventually too?

And that's likely something we'll want to do in 3.2 too as usually Google makes this a requirement for new apps at some point. Judging by previous years, that would be from ~August 2021 (not announced yet): https://support.google.com/googleplay/android-developer/answer/113469#targetsdk

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to resolve #38913 before we can bump the targetSdk and compileSdk as the current workaround we're using is ignored when targeting Android 11 (API level 30).

@akien-mga akien-mga merged commit a4f70d7 into godotengine:master Oct 29, 2020
@akien-mga
Copy link
Member

Thanks!

@m4gr3d m4gr3d deleted the update_android_plugins branch October 29, 2020 15:37
@akien-mga
Copy link
Member

Cherry-picked for 3.2.4.

@akien-mga akien-mga added enhancement and removed cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Nov 11, 2020
@m4gr3d
Copy link
Contributor Author

m4gr3d commented Nov 11, 2020

@akien-mga Is there a separate 3.2.4 branch? I don't see the changes made here in the 3.2 branch.

@akien-mga
Copy link
Member

@m4gr3d I stage them locally and push them all as a batch after testing: #43456.

@m4gr3d
Copy link
Contributor Author

m4gr3d commented Nov 11, 2020

@akien-mga Sounds good. I thought you had already done that step hence my confusion :)

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

Successfully merging this pull request may close these issues.

2 participants