-
Notifications
You must be signed in to change notification settings - Fork 30
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
Prepare Gradle 9.0 deprecations #177
Conversation
Motivation: Starting with Gradle 9.0, Gradle will not run builds if a project directory is missing or read-only. (ref: https://docs.gradle.org/8.7/userguide/upgrading_version_8.html#deprecated_missing_project_directory) Modification: - Create project directories Result: - Deprecation for Gradle 9.0 is prepared for this behavior
fdaf11d
to
8b7893a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! 👍 👍 👍
Let me merge this after build with Gradle 9.0, which isn't release yet, to make sure if we don't miss anything.
The `AbstractArchiveTask.archivePath` property has been deprecated. This is scheduled to be removed in Gradle 9.0. https://docs.gradle.org/8.7/dsl/org.gradle.api.tasks.bundling.AbstractArchiveTask.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 👍 👍
@sh-cho Thanks! 😄 |
relocatedProjects.each { | ||
if (it != project && !it.hasFlags('no_aggregation')) { | ||
injars it.tasks.shadedJar.archivePath | ||
injars it.tasks.shadedTestJar.archivePath | ||
injars it.tasks.shadedJar.aarchiveFile.get().asFile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😨
Motivation: Starting with Gradle 9.0, Gradle will not run builds if a project directory is missing or read-only. (ref: https://docs.gradle.org/8.7/userguide/upgrading_version_8.html#deprecated_missing_project_directory) Modification: - Create project directories under `build/virtual-projects` - `:bom`, `:version-catalog` - `:dependencyManagement` will be handled via line/gradle-scripts#177 Result: - Deprecation for Gradle 9.0 is prepared for this behavior
Motivation:
Prepare deprecations which will be removed or fail in Gradle 9.0
Modification:
build/virtual-projects
. Starting 9.0, Gradle will not run builds if a project directory is missing or read-only. (https://docs.gradle.org/8.7/userguide/upgrading_version_8.html#deprecated_missing_project_directory)base { }
(https://docs.gradle.org/8.7/userguide/upgrading_version_8.html#base_convention_deprecation)AbstractArchiveTask.archivePath
witharchiveFile
(https://docs.gradle.org/8.7/javadoc/org/gradle/api/tasks/bundling/AbstractArchiveTask.html#getArchivePath--)Result: