RFR: JDK-8215447: Investigate if current implementation of --license-file is correct for RPM packagesI

2019-08-13 Thread Alexey Semenyuk

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).


- Install license file in the correct location in stead of the app's 
installation directory, update corresponding jtreg tests.


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

[2] http://cr.openjdk.java.net/~asemenyuk/8215447/webrev.00/

Thanks,
Alexey



Re: RFR: JDK-8215447: Investigate if current implementation of --license-file is correct for RPM packagesI

2019-08-13 Thread Andy Herrick

looks good

/Andy

On 8/13/2019 3:10 PM, Alexey Semenyuk 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).


- Install license file in the correct location in stead of the app's 
installation directory, update corresponding jtreg tests.


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

[2] http://cr.openjdk.java.net/~asemenyuk/8215447/webrev.00/

Thanks,
Alexey



Re: RFR: JDK-8215447: Investigate if current implementation of --license-file is correct for RPM packagesI

2019-08-13 Thread Alexander Matveev

Looks good.

On 8/13/2019 12:10 PM, Alexey Semenyuk 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).


- Install license file in the correct location in stead of the app's 
installation directory, update corresponding jtreg tests.


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

[2] http://cr.openjdk.java.net/~asemenyuk/8215447/webrev.00/

Thanks,
Alexey