Skip to content

Gradle 8.3 and AGP 7.3.1#752

Merged
RBusarow merged 1 commit intomainfrom
rick/gradle-agp
Sep 28, 2023
Merged

Gradle 8.3 and AGP 7.3.1#752
RBusarow merged 1 commit intomainfrom
rick/gradle-agp

Conversation

@RBusarow
Copy link
Copy Markdown
Contributor

The AGP update becomes necessary in order to resolve:

Could not determine the dependencies of task ':sample:app:testDebugUnitTest'.
> Could not create task ':sample:app:processDebugResources'.
   > Cannot use @TaskAction annotation on method IncrementalTask.taskAction$gradle_core() because interface org.gradle.api.tasks.incremental.IncrementalTaskInputs is not a valid parameter to an action method.


[versions]
agp = "7.1.0"
agp = "7.3.1"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Updating to AGP 8 should resolve the lint job failure, but that will also require use of JDK 17.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The lint issue was easy enough to fix, since it's just a sample app.

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-7.6-bin.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-8.3-all.zip
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

bin is good enough

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Technically, but it's a pain in the butt when you're doing Gradle things and want to click into the source or even just read the javadoc.

Since Gradle's not being bundled into the release, it doesn't do any harm to just improve the devx a little bit.

The AGP update becomes necessary in order to resolve:

```
Could not determine the dependencies of task ':sample:app:testDebugUnitTest'.
> Could not create task ':sample:app:processDebugResources'.
   > Cannot use @TaskAction annotation on method IncrementalTask.taskAction$gradle_core() because interface org.gradle.api.tasks.incremental.IncrementalTaskInputs is not a valid parameter to an action method.
```
@RBusarow
Copy link
Copy Markdown
Contributor Author

I would like to replace the references to the deprecated com.android.build.gradle.api.BaseVariant and its subtypes in AnvilPlugin, but that stuff should really be backed by integration tests. So, coming soon™.

@RBusarow RBusarow marked this pull request as ready for review September 28, 2023 19:50
@RBusarow RBusarow merged commit a2a3ada into main Sep 28, 2023
@RBusarow RBusarow deleted the rick/gradle-agp branch September 28, 2023 19:51
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