RFR: JDK-8141444 Clean up building of JDK launchers
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Thu Nov 5 09:55:35 UTC 2015
On 2015-11-05 08:58, Erik Joelsson wrote:
> This is just so nice! Only one small comment. In LauncherCommon.gmk
> there are two ifeq for OS macosx. Perhaps those should be combined?
Yep, that's a good idea. I didn't notice that when I moved the
PACKAGE_PATH logic.
I just combined the two and didn't respin the webrev. I hope you don't
complain. :)
/Magnus
>
> /Erik
>
> On 2015-11-05 08:27, Magnus Ihse Bursie wrote:
>> The JDK launchers have been built by a macro which is using
>> positional arguments instead of named argument. This needs to be
>> fixed to be able to properly track what is happening.
>>
>> Some additional TLC for launchers is also needed.
>>
>> To verify the fix, I've used the new COMPARE_BUILD feature.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8141444
>> WebRev:
>> http://cr.openjdk.java.net/~ihse/JDK-8141444-clean-up-launchers/webrev.01
>>
>> /Magnus
>>
>
More information about the build-dev
mailing list