> On May 11, 2016, at 2:21 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
> 
>> 
>> On May 11, 2016, at 10:17 AM, Jim Laskey (Oracle) <james.las...@oracle.com> 
>> wrote:
>> 
>> 
>>> On May 11, 2016, at 2:03 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
>>> 
>>> 
>>>> On May 9, 2016, at 10:41 AM, Jim Laskey (Oracle) <james.las...@oracle.com> 
>>>> wrote:
>>>> 
>>>> Modifies the jlink sort-plugin to accept a classlist to order files in the 
>>>> jimage.
>>>> 
>>>> http://cr.openjdk.java.net/~jlaskey/8155237/webrev/index.html
>>>> https://bugs.openjdk.java.net/browse/JDK-8155237
>>> 
>>> This looks good and also cleaned up the original code.  I saw that it’s 
>>> pushed which is fine. 
>>> 
>>> A small comment: this plugin should throw error if @file not exists.
>> 
>> This is deliberate, since the classlist may not be there during one phase of 
>> the build.  We could issue a warning.
>> 
> 
> At build time, it should know the image is created using classlist or not and 
> so the —-order-resources option can only be specified when it’s present.  
> This would help catching issue at link time in case if anything goes wrong.


Erik, comments?

> 
>> 
>>> Does jimage list command show the jimage entries in alphabetically order?  
>>> It would be useful to provide an option to list the entries as layout in 
>>> the jimage file (a RFE).
>> 
>> Yes to alphabetically.  Please post an RFE for file order.
>> 
> 
> OK.
> 
> Mandy

Reply via email to