On Wed, 24 Sep 2025 03:22:16 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
>> **Issue**: >> Below two Gradle 9.0.0 specific warnings are observed with current grade >> 8.14.2 >> 1. The `Project.exec(Closure)` method has been deprecated. This is scheduled >> to be removed in Gradle 9.0 >> 2. Using method `javaexec(Closure)` has been deprecated. This is scheduled >> to be removed in Gradle 9.0. >> >> Both these methods are removed in Gradle 9.0.0. >> Hence they should be replaced with appropriate APIs. >> Refer: >> https://docs.gradle.org/8.14.2/userguide/upgrading_version_8.html#deprecated_project_exec >> >> **Fix**: >> Both warnings can be fixed by replacing the calls with` >> ExecOperations.exec(Action)`. >> There are two different scenarios where we use the above two APIs. >> - Gradle files >> - Groovy class files >> => In Both the scenarios, we have to use the `ExecOperations.exec(Action)`, >> but the approach differs. >> >> 1. Gradle file >> - The calls are simply replaced as, >> * `Project.exec {` is replaced with `execOps.exec { ExecSpec spec ->` >> * `Project.javaexex {` is replaced with `execOps.javaexec { >> JavaExecSpec spec ->` >> 2. For Groovy classes >> - `ExecOperations` needs to be **injected** using `@Inject` tag >> 1. `NativeCompileTask` class >> - In the `NativeCompileTask` class, a method `execCompile()` is >> added which executes an action. >> - The child classes of `NativeCompileTask`, use this `execCompile()` >> method. >> 2. Other classes are not related to each other. They independently >> execute the respective action. >> >> **Verification**: >> 1. Run all gradle tasks with **—warning-mode all** option, with gradle >> 8.14.2 >> 2. No Gradle 9.0.0 specific warning is seen in build log >> 4. Build/all tasks complete successfully. > > Ambarish Rapte has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains two additional > commits since the last revision: > > - Merge branch 'master' into gradle-900-warnings > - replace project.exec and project.javaexec Reviewers: @kevinrushforth @tiainen ------------- PR Comment: https://git.openjdk.org/jfx/pull/1918#issuecomment-3328104282
