> On 11 Jan 2017, at 23:47, Mandy Chung <[email protected]> wrote: > > Webrev: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8160286/webrev.00/
Looks fine. It’s good to share as much of this code as possible between these packaging tools. I didn’t look too hard at Graph and TopoSorter, these look similar to what is in other areas, e.g. jdeps, In the jar option parsing did you mean to construct ModulePath with true? This will support jmod files. -Chris.
