Re: RFR: JDK-8231279: Change install location for copyright file for Debian package

2019-09-23 Thread Alexey Semenyuk
On 9/23/2019 6:05 PM, Alexander Matveev wrote: Hi Alexey, http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxDebBundler.java.frames.html Maybe add new line between 41-42 and remove 43 to align import static with other static

Re: RFR: JDK-8231279: Change install location for copyright file for Debian package

2019-09-23 Thread Alexander Matveev
Hi Alexey, http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxDebBundler.java.frames.html Maybe add new line between 41-42 and remove 43 to align import static with other static imports. Looks fine. Thanks, Alexander On

Re: RFR: JDK-8231279: Change install location for copyright file for Debian package

2019-09-23 Thread Andy Herrick
looks good. /Andy On 9/23/2019 7:49 AM, 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). Baseline for the fix is [3] patch. [1]

RFR: JDK-8231279: Change install location for copyright file for Debian package

2019-09-23 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). Baseline for the fix is [3] patch. [1] https://bugs.openjdk.java.net/browse/JDK-8231279 [2] http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/