On Wed, 12 May 2021 17:02:27 GMT, Crazyjavahacking <github.com+1445818+crazyjavahack...@openjdk.org> wrote:
>> This PR fixes the gradle deprecation warnings described in >> [JDK-8240336](https://bugs.openjdk.java.net/browse/JDK-8240336) and updates >> the JavaFX build to use gradle 7.0.1 as described in >> [JDK-8263760](https://bugs.openjdk.java.net/browse/JDK-8263760). The minimum >> version of gradle is set to 6.3. >> >> In addition to keeping gradle up to date, updating to gradle 7 will allow >> building and testing JavaFX with JDK 16. >> >> I have done a full build and test on all three platforms, comparing the >> artifacts produced before and after this change. >> >> ### Notes to Reviewers: >> >> The PR branch has two separate commits, one for each fix, in case reviewers >> want to look at them separately. As always, they will be squashed into a >> single commit when integrated. Both bug IDs will be listed in the commit. >> >> The following changes were done for >> [JDK-8240336](https://bugs.openjdk.java.net/browse/JDK-8240336) to eliminate >> the use of deprecated features removed in gradle 7: >> >> 1. Replaced `compile` with either `implementation` or `compileClasspath` as >> needed >> 2. Replaced obsolete `archiveName` and `destinationDir` properties in >> archive tasks with `archiveFileName` and `destinationDirectory` >> 3. Added missing `@Input` annotation to custom Groovy task properties >> 4. Bumped the minimum version of gradle to 6.3 (which we have been using for >> more than 1 year) >> >> >> The following changes were done for >> [JDK-8263760](https://bugs.openjdk.java.net/browse/JDK-8263760) to update to >> gradle 7.0.1: >> >> 1. Ran `bash ./gradlew wrapper --gradle-version=7.0.1` >> 2. Updated the gradle version in `build.properties` to `7.0.1` >> 3. Updated the SHA-256 checksum in `gradle/wrapper/gradle-wrapper.properties` > > gradlew line 47: > >> 45: >> 46: # Add default JVM options here. You can also use JAVA_OPTS and >> GRADLE_OPTS to pass JVM options to this script. >> 47: DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"' > > This is automatically generated by Gradle, but the heap memory seems to be > too low. > > + the same for `gradlew.bat` I agree with this concern, and will revert this auto-generated change. I don't know why the gradle team felt that a 64Mbyte heap was sufficient. ------------- PR: https://git.openjdk.java.net/jfx/pull/498
