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. This pull request has now been integrated. Changeset: 4af67c12 Author: Nir Lisker <nlis...@openjdk.org> URL: https://git.openjdk.org/jfx/commit/4af67c1286f119a967e2758ab373c1a625843308 Stats: 97 lines in 1 file changed: 5 ins; 77 del; 15 mod 8344906: Simplify Java version parsing in the build file Reviewed-by: kcr, arapte ------------- PR: https://git.openjdk.org/jfx/pull/1647
