> On Apr 4, 2017, at 12:42 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > > On 03/04/2017 19:41, Mandy Chung wrote: > >> Webrev: >> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8175819/webrev.00/ >> > I went through the updates to jlink, assuming test SystemModulesTest will be > aligned to the recent mails. >
It’s already aligned. I updated webrev.00 in place yesterday. Maybe you need to reload it if you happened opening the old version. > In DefaultImageBuilder.storeFiles then > map(ResourcePoolModule::osName).orElse(null) would be cleaner. > OK. > I'm sure Volker or someone maintaining the AIX port will ask for > jdk.tools.jlink.internal.Platform to be extended to handle that platform. I was wondering if Volker would ask that. Anyway, I have added Platform::AIX constant. Will send out a new webrev. Mandy
