chore(deps): update dependency gradle to v8.14.2#9
Conversation
WalkthroughThe Gradle wrapper scripts and configuration have been updated. The Gradle version in the properties file was increased, with new timeout and validation options added. The shell and batch scripts were revised for improved POSIX compliance, quoting, error handling, and Java invocation, with updated copyright and license headers. Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
gradlew.bat (1)
73-78: Remove the superfluous ‑classpath argument
java -jarignores any preceding-classpath/-cpoptions.
Passing an empty class-path (-classpath "") is therefore unnecessary and may trigger a benign “option ignored” warning on some JVMs.-@rem Execute Gradle -"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %* +@rem Execute Gradle +"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %*Dropping the flag keeps the command line cleaner.
gradlew (3)
117-118: Double-escaped empty CLASSPATH is hard to read
CLASSPATH="\\\"\\\""eventually becomes""so that the JVM receives an empty string.
A simpler, more self-documenting approach is:CLASSPATH=and later pass
-classpath ""explicitly.
Functionally equivalent, but much easier to maintain.
145-162: ulimit section may fail on shells lacking-n
ulimit -H -n/ulimit -nare not POSIX‐mandated and are missing on some busybox or dash builds.
You already guard for Cygwin/Darwin/NonStop, but consider feature-testing the command instead:if command -v ulimit >/dev/null 2>&1; then # existing logic … fiThis avoids emitting misleading warnings on minimal shells.
220-224: Graceful degradation whenxargsis missingThe script aborts if
xargsis absent. On some Alpine or embedded images, only busybox is available and may lackxargs.
If portability is a concern, you could fall back to the previous splitting logic instead of outright failure.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
gradle/wrapper/gradle-wrapper.jaris excluded by!**/*.jar
📒 Files selected for processing (3)
gradle/wrapper/gradle-wrapper.properties(1 hunks)gradlew(3 hunks)gradlew.bat(1 hunks)
🔇 Additional comments (3)
gradle/wrapper/gradle-wrapper.properties (1)
3-5: Validate distribution integrity and timeout valueThe new
validateDistributionUrl=trueflag only checks that the URL points toservices.gradle.org, but it does not protect against tampering.
Consider adding the SHA-256 checksum that Gradle 8+ supports (distributionSha256Sum=<hash>), so the wrapper verifies the downloaded archive’s integrity as well.In addition,
networkTimeout=10000(10 s) can be tight for CI runners behind slow or proxied connections. If your pipeline occasionally downloads the wrapper from scratch, you might want to bump this to 30000 ms or rely on the default (30 s).gradlew.bat (1)
86-89: Good catch capturing non-zero ERRORLEVELStoring
ERRORLEVELinEXIT_CODEand forcing it to a minimum of 1 prevents silent success exits―nicely done.gradlew (1)
205-206: 64 MiB heap might be too restrictive for modern GradleGradle 8.14 performs more byte-code scanning and Kotlin DSL evaluation. Several Android projects now require at least 256 MiB for configuration.
Unless you have compelling memory constraints, raising the defaults improves reliability:-DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"' +DEFAULT_JVM_OPTS='"-Xmx512m" "-Xms256m"'
1 participant
This PR contains the following updates:
8.11.1->8.14.2Release Notes
gradle/gradle (gradle)
v8.14.2Compare Source
v8.14.1Compare Source
v8.14Compare Source
v8.13Compare Source
v8.12.1: 8.12.1Compare Source
The Gradle team is excited to announce Gradle 8.12.1.
Read the Release Notes
We would like to thank the following community members for their contributions to this release of Gradle:
Abhiraj Adhikary,
Ayush Saxena,
Björn Kautler,
davidburstrom,
Dominic Fellbaum,
Emmanuel Ferdman,
Finn Petersen,
Johnny Lim,
Mahdi Hosseinzadeh,
Martin Bonnin,
Paint_Ninja,
Petter Måhlén,
Philip Wedemann,
stegeto22,
Tanish,
TheGoesen,
Tim Nielens,
Trout Zhang,
Victor Merkulov
Upgrade instructions
Switch your build to use Gradle 8.12.1 by updating your wrapper:
See the Gradle 8.x upgrade guide to learn about deprecations, breaking changes and other considerations when upgrading.
For Java, Groovy, Kotlin and Android compatibility, see the full compatibility notes.
Reporting problems
If you find a problem with this release, please file a bug on GitHub Issues adhering to our issue guidelines.
If you're not sure you're encountering a bug, please use the forum.
We hope you will build happiness with Gradle, and we look forward to your feedback via Twitter or on GitHub.
v8.12: 8.12Compare Source
The Gradle team is excited to announce Gradle 8.12.
Read the Release Notes
We would like to thank the following community members for their contributions to this release of Gradle:
Abhiraj Adhikary,
Ayush Saxena,
Björn Kautler,
davidburstrom,
Dominic Fellbaum,
Emmanuel Ferdman,
Finn Petersen,
Johnny Lim,
Mahdi Hosseinzadeh,
Martin Bonnin,
Paint_Ninja,
Petter Måhlén,
Philip Wedemann,
stegeto22,
Tanish,
TheGoesen,
Tim Nielens,
Trout Zhang,
Victor Merkulov
Upgrade instructions
Switch your build to use Gradle 8.12 by updating your wrapper:
See the Gradle 8.x upgrade guide to learn about deprecations, breaking changes and other considerations when upgrading.
For Java, Groovy, Kotlin and Android compatibility, see the full compatibility notes.
Reporting problems
If you find a problem with this release, please file a bug on GitHub Issues adhering to our issue guidelines.
If you're not sure you're encountering a bug, please use the forum.
We hope you will build happiness with Gradle, and we look forward to your feedback via Twitter or on GitHub.
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.