Re: RFR: JDK-8230651: Use version string from main module

2019-09-23 Thread Andy Herrick

Looks good.

/Andy

On 9/20/2019 5:54 PM, Alexander Matveev wrote:

Please review the jpackage fix for bug [1] at [2].

This is a fix for the JDK-8200758-branch branch of the open sandbox 
repository (jpackage).


- Version from main module (if exist) will be used as --app-version if 
--app-version is not specified.


[1] https://bugs.openjdk.java.net/browse/JDK-8230651

[2] http://cr.openjdk.java.net/~almatvee/8230651/webrev.01/

Thanks,
Alexander


Re: RFR: JDK-8230651: Use version string from main module

2019-09-20 Thread Alexey Semenyuk

Looks good.

Unfortunately the new test helper classes don't support --module option 
yet and the test can not be implemented based on them.


- Alexey

On 9/20/2019 5:54 PM, Alexander Matveev wrote:

Please review the jpackage fix for bug [1] at [2].

This is a fix for the JDK-8200758-branch branch of the open sandbox 
repository (jpackage).


- Version from main module (if exist) will be used as --app-version if 
--app-version is not specified.


[1] https://bugs.openjdk.java.net/browse/JDK-8230651

[2] http://cr.openjdk.java.net/~almatvee/8230651/webrev.01/

Thanks,
Alexander




RFR: JDK-8230651: Use version string from main module

2019-09-20 Thread Alexander Matveev

Please review the jpackage fix for bug [1] at [2].

This is a fix for the JDK-8200758-branch branch of the open sandbox 
repository (jpackage).


- Version from main module (if exist) will be used as --app-version if 
--app-version is not specified.


[1] https://bugs.openjdk.java.net/browse/JDK-8230651

[2] http://cr.openjdk.java.net/~almatvee/8230651/webrev.01/

Thanks,
Alexander