Review request for generating debian package for jdk modules

Mandy Chung Mandy.Chung at Sun.COM
Mon Feb 15 16:24:09 PST 2010


On 02/15/10 15:43, Dalibor Topic wrote:
> Mandy Chung wrote:
>> This change will generate debian packages for jdk modules.
>>
>> Webrev at:
>>   http://cr.openjdk.java.net/~mchung/jigsaw/jigsaw-deb-pkgs/
>>
>> Summary:
>> 1. Include unpack200 in the jdk.boot module depended by
>>   the install script to uncompress the jpg generated packages.
> 
> Looks good as well.
>> 3. BuildPackages.gmk is the makefile for building packages.
>>   Currently it checks if it's running on debian systems or not.
>>   If not, no package will be generated.  This gmk file will   be
>> modified when jpkg is extended to support other native
>>   packaging format or module neutral format.
> 
> Small nit: you'll only need to pass -c to jpkg if there is 
> something with a main class in a module to run - I assume that's 
> not the case for the jdk modules? Otherwise, it looks fine, too.

Right.  -c option is not needed for jdk modules.  I will take that out.

Thanks for the review, Dalibor.
Mandy

> cheers,
> dalibor topic
> 



More information about the jigsaw-dev mailing list