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

Alexey Semenyuk alexey.semenyuk at oracle.com
Mon Sep 23 22:08:53 UTC 2019



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 imports.
This is how Netbeans manages imports. I don't touch this part of the 
code, just click on "Fix Imports" menu item and magic happens :)

- Alexey
>
> Looks fine.
>
> Thanks,
> Alexander
>
> On 9/23/2019 4: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] https://bugs.openjdk.java.net/browse/JDK-8231279
>>
>> [2] http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/
>>
>> [3] http://cr.openjdk.java.net/~asemenyuk/8231277/webrev.00/
>>
>



More information about the core-libs-dev mailing list