On Sat, 23 Nov 2024 09:22:36 GMT, Nir Lisker <nlis...@openjdk.org> wrote:
> Replaces the manual versions handling with > [Version](https://docs.oracle.com/en/java/javase/23/docs/api/java.base/java/lang/Runtime.Version.html). > > Changes: > * Removed the methods `parseJavaVersion(String)`, `parseJdkVersion(String)` > and `compareJdkVersion(String, String)`, and replaced them with > `Version.parse` and `Version.compareTo` methods. > * Removed the build properties of the Java version on which the Gradle build > runs because they are only used for logging. Replaced with logging them > directly. Note that Gradle logs by itself the info of the runtime it uses, so > there's no need to logs these manually. > * Simplified the build JDK version by working directly with `Version` instead > of a `String`. This allows to store less build properties. Also used > try-with-resources to close the stream. Note that the whole build JDK manual > invocation hack should be replaced with the Java Toolchain. > * Simplified the `verifyJava` task, which could also be replaced with > built-in Gradle tools. Yes, `Version` doesn't parse the old 1.x JDK formats. Since the build JDK needs to be up to date anyway, I didn't see it as a problem. ------------- PR Comment: https://git.openjdk.org/jfx/pull/1647#issuecomment-2499772605
