On 02/08/2016 04:39, Sundararajan Athijegannathan wrote:

Please review http://cr.openjdk.java.net/~sundar/8159487/webrev.00/ for
https://bugs.openjdk.java.net/browse/JDK-8159487

OS_NAME, OS_ARCH, OS_VERSION properties are already added due to another
fix. Just adding "JAVA_VERSION" and a test change to check these
properties exist in release file.

This seems to put the value of the "java.version" property into the release file, I don't think you want that. Consider the cross targeting case where jlink is running on 9 but the modules for the target image are 9.1. So it needs to come from the java.base module for the target run-time image.

-Alan

Reply via email to