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

Upgrade gradle version to 8.0.1 #6656

Merged
merged 7 commits into from
Feb 20, 2023
Merged

Upgrade gradle version to 8.0.1 #6656

merged 7 commits into from
Feb 20, 2023

Conversation

eddumelendez
Copy link
Member

@eddumelendez eddumelendez commented Feb 17, 2023

From Gradle 8.0 release notes

Gradle 7.6 introduced toolchain repositories for increased flexibility. In Gradle 8.0, there is no longer a default toolchain provisioner. You have to declare at least one Java Toolchain repository explicitly. This can be done via toolchain repository plugins, like the Foojay Toolchains Plugin:

plugins {
    id("org.gradle.toolchains.foojay-resolver-convention") version("0.4.0")
}

BuildScan was shared with the Gradle Team in order to know what's happening.

@eddumelendez eddumelendez requested a review from a team as a code owner February 17, 2023 18:15
Signed-off-by: gradle-update-robot <gradle-update-robot@regolo.cc>
Signed-off-by: gradle-update-robot <gradle-update-robot@regolo.cc>
@github-actions github-actions bot added the gradle-wrapper Pull requests that update Gradle wrapper label Feb 20, 2023
@eddumelendez eddumelendez changed the title Add foojay-toolchains plugin Upgrade gradle version to 8.0.1 Feb 20, 2023
@eddumelendez
Copy link
Member Author

new issue:

Possible solutions:
      1. Declare task ':test-support:jar' as an input of ':test-support:test'.
      2. Declare an explicit dependency on ':test-support:jar' from ':test-support:test' using Task#dependsOn.
      3. Declare an explicit dependency on ':test-support:jar' from ':test-support:test' using Task#mustRunAfter.

By removing the test-support dependency from modules the error is gone. I can investigate later

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.

None yet

2 participants