On Tue, 10 Sep 2024 08:13:44 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
>> Following 2 warnings are observed with the JavaFX build. >> These warnings need to be addressed for updating to Gradle 9.0 in future. >> >> 1. The org.gradle.api.plugins.JavaPluginConvention type has been deprecated. >> This is scheduled to be removed in Gradle 9.0. Consult the upgrading guide >> for further information: >> https://docs.gradle.org/8.9/userguide/upgrading_version_8.html#java_convention_deprecation >> `=>>` This warning gets fixed by the change on line number 2111 in >> build.gradle >> >> 2. The automatic loading of test framework implementation dependencies has >> been deprecated. This is scheduled to be removed in Gradle 9.0. Declare the >> desired test framework directly on the test suite or explicitly declare the >> test framework implementation dependencies on the test's runtime classpath. >> Consult the upgrading guide for further information: >> https://docs.gradle.org/8.9/userguide/upgrading_version_8.html#test_framework_implementation_dependencies >> `=>>` This warning gets fixed by the change on line number 2128 in >> build.gradle. It also requires the verification metadata to be updated. >> >> Verification: >> 1. Verify that build completes successfully >> 2. Above two warnings are not observed in build log. >> 3. This warning message is NOT observed in build log: Deprecated Gradle >> features were used in this build, making it incompatible with Gradle 9.0. > > Ambarish Rapte has updated the pull request incrementally with one additional > commit since the last revision: > > fix for windows warning Updated the PR with fix for windows warning: [commit](https://github.com/openjdk/jfx/pull/1559/commits/3c8cbc85ea8666574c43e3d6cb932b5ef32ffc1a) Warnings are : - Build file 'C:\ade\jfx\rt\build.gradle': line 4729 The CopyProcessingSpec.setDirMode(Integer) method has been deprecated. This is scheduled to be removed in Gradle 9.0. Please use the dirPermissions(Action) method instead. Consult the upgrading guide for further information: https://docs.gradle.org/8.9/userguide/upgrading_version_8.html#unix_file_permissions_deprecated - Build file 'C:\ade\jfx\rt\build.gradle': line 4730 The CopyProcessingSpec.setFileMode(Integer) method has been deprecated. This is scheduled to be removed in Gradle 9.0. Please use the filePermissions(Action) method instead. Consult the upgrading guide for further information: https://docs.gradle.org/8.9/userguide/upgrading_version_8.html#unix_file_permissions_deprecated [CopyProcessingSpec.setDirMode(Integer)](https://docs.gradle.org/8.9/javadoc/org/gradle/api/file/CopyProcessingSpec.html#setDirMode(java.lang.Integer)) and [CopyProcessingSpec.setFileMode(Integer)](https://docs.gradle.org/8.9/javadoc/org/gradle/api/file/CopyProcessingSpec.html#setFileMode(java.lang.Integer)) are both deprecated and should be replaced with [dirPermissions(Action)](https://docs.gradle.org/8.9/javadoc/org/gradle/api/file/CopyProcessingSpec.html#dirPermissions(org.gradle.api.Action)) & [filePermissions(Action)](https://docs.gradle.org/8.9/javadoc/org/gradle/api/file/CopyProcessingSpec.html#filePermissions(org.gradle.api.Action)) respectively. Using [unix](https://docs.gradle.org/8.9/javadoc/org/gradle/api/file/ConfigurableFilePermissions.html#unix(int)) method to achieve this seems like a good way. ------------- PR Comment: https://git.openjdk.org/jfx/pull/1559#issuecomment-2340053410
