Re: RFR: JDK-8220807: excessive verbose output when running with --win-per-user-install

2019-07-15 Thread Alexey Semenyuk

Looks good.

- Alexey

On 7/15/2019 7:48 AM, Andy Herrick wrote:
revised [3] to only suppress these warnings when 
--win-per-user-install option used


[3] http://cr.openjdk.java.net/~herrick/8220807/webrev.02/

/Andy

On 7/12/2019 1:22 PM, Alexey Semenyuk wrote:

Andy,

The fix suppresses warning for system wide installations too,not just 
for per user. Is this by intention?


- Alexey

On 7/12/2019 10:31 AM, Andy Herrick 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).


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

[2] http://cr.openjdk.java.net/~herrick/8220807/

/Andy







Re: RFR: JDK-8220807: excessive verbose output when running with --win-per-user-install

2019-07-15 Thread Andy Herrick
revised [3] to only suppress these warnings when --win-per-user-install 
option used


[3] http://cr.openjdk.java.net/~herrick/8220807/webrev.02/

/Andy

On 7/12/2019 1:22 PM, Alexey Semenyuk wrote:

Andy,

The fix suppresses warning for system wide installations too,not just 
for per user. Is this by intention?


- Alexey

On 7/12/2019 10:31 AM, Andy Herrick 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).


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

[2] http://cr.openjdk.java.net/~herrick/8220807/

/Andy





Re: RFR: JDK-8220807: excessive verbose output when running with --win-per-user-install

2019-07-12 Thread Alexey Semenyuk

Andy,

The fix suppresses warning for system wide installations too,not just 
for per user. Is this by intention?


- Alexey

On 7/12/2019 10:31 AM, Andy Herrick 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).


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

[2] http://cr.openjdk.java.net/~herrick/8220807/

/Andy