Upgrade to Gradle 8.10.2 and AGP 8.8.2#2264
Merged
jkasten2 merged 6 commits intoidentity_verification_betafrom Mar 4, 2025
Merged
Upgrade to Gradle 8.10.2 and AGP 8.8.2#2264jkasten2 merged 6 commits intoidentity_verification_betafrom
jkasten2 merged 6 commits intoidentity_verification_betafrom
Conversation
Commit just contains the upgrades themselves, a follow up commit will includes changes required to address their build requirements.
* namespace is now always required * Some gradle.properties values were completely dropped in APG 8 * Publish release access changed to avoid build error, more changes are need to actually publish.
Since Gradle / AGP 8 upgrade the example doesn't work for Huawei builds, however this uncovered we were not correctly omitting it for all tasks. We will fix Huawei in the example project not building in follow up commit.
The top level gradle project needs a Java / Kotlin module, otherwise it skipped publishing an .aar file. While there is no code, projects consuming this SDK look for it. * Maybe we can omit the .aar on in the future for this top level project but keeping it working the same for now. Built SDK with Java 17 and tested publishing with a staging build with: * Java 11 * Gradle 7.3.3 * APG 7.2.2 * Android 5 & 14 Emulator WARNING: It is still recommended building with Java 17 instead of 21 when publishing, as those on older version of R8 will get this build error: * Cannot invoke "String.length()" because "<parameter1>" is null * See https://stackoverflow.com/q/77788162
jinliu9508
approved these changes
Mar 4, 2025
Contributor
jinliu9508
left a comment
jinliu9508
left a comment
There was a problem hiding this comment.
Build was failed until AGP is upgraded. After Gradle and AGP is upgraded, was able to successfully build and test with Android Build Tools v35, Java 11.
18 tasks
|
@jkasten2 Fyi:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
One Line Summary
Upgrade to latest production AGP and it's supported Gradle version.
Details
Motivation
There are two reasons:
Scope
Just updated what is needed to be able build, run, and publish with the newest AGP version.
Testing
Unit testing
N/A
Manual testing
Built SDK with Java 17 (Note: do NOT publish with JDK21 yet) and tested publishing with a staging build with:
Affected code checklist
Checklist
Overview
Testing
identity_verification_betabranch is behind main. Created another branch to prove the tests work after Gradle / AGP upgrade: Temp - PR to run CI tests for the upgrade/gradle-and-agp-8 #2265Final pass
This change is