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 Gradle & AGP to version 8 #575

Merged
merged 7 commits into from
Feb 23, 2024
Merged

Update Gradle & AGP to version 8 #575

merged 7 commits into from
Feb 23, 2024

Conversation

oakkitten
Copy link
Collaborator

Needed to go forward. See individual commit messages.

The changes in gradle.properties were made by the migration tool.

This also disables the weaving of Cats, as transform API were disabled
in AGP. Note that somehow the classes are still present, but actually
triggering transformation results in an error.

Cat weaving is fixed in a different way in the next commit.
We should be really moving away from AspectJ. However, for now there are
no ready replacements for Cats, unless we are willing to get rid of
annotation-based logging. Perhaps in the future Krang could be an option.

Meanwhile, Android AspectJ Gradle Plugin seems to be actually working
with AGP 8. Unfortunately, it is no longer maintained, but hopefully it can
serve us at least until the next major version bump of AGP.

AAGP: https://github.com/Ibotta/gradle-aspectj-pipeline-plugin
Krang: https://github.com/milis92/Krang
This is obsolete with AGP 8
No longer needed as EventBus apparently supports AndroidX since 3.3
This fixes the following warning warning:

The option setting
'android.defaults.buildfeatures.buildconfig=true' is
deprecated. The current default is 'false'. It will be
removed in version 9.0 of the Android Gradle plugin.
You can resolve this warning in Android Studio via
`Refactor` > `Migrate BuildConfig to Gradle Build
Files`
Copy link
Collaborator

@zopieux zopieux left a comment

Choose a reason for hiding this comment

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

I'm a simple man. I see less code, I click Approve.

Note however I know nothing of gradle & even less the dark magic Android does with it.

@oakkitten oakkitten merged commit da63de3 into master Feb 23, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants