> 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. > 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. — Jim
