Review request: jar tool to transform a plain jar file to a modular jar

Alan Bateman Alan.Bateman at oracle.com
Mon Sep 12 06:06:34 PDT 2011


Mandy Chung wrote:
>  Updated webrev:
>     http://cr.openjdk.java.net/~mchung/jigsaw/webrevs/modularize-jar.01/
I see Sherman has done a detailed review on the sun.tools.jar changes.

I'm looking at additions to java.util.jar.JarFile and I'm wondering if 
the class or method descriptions should having wording to explain that a 
JAR file can have a legacy manifest and also a module-info. I also 
wonder if we will need to update the JAR specification too. Minor 
comment is that the new methods should have @since 1.8.

A passing comment on SimpleLibrary lines 980-988 is that they could be 
replaced with Files.write(is, md.toPath().resolve("info"));

The sun.tools.jar.ModuleInfo class looks very useful and I'm sure we 
will have other places that need to generate module-info classes. Would 
it make sense to put it in org.openjdk.jigsaw? If so then it would be 
good to have it support requires optional too.

-Alan.



More information about the jigsaw-dev mailing list