Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

2016-11-23 Thread Mandy Chung
> On Nov 23, 2016, at 6:05 AM, Alan Bateman wrote: > > > > On 22/11/2016 21:07, Mandy Chung wrote: >> This patch moves src.zip and jrt-fs.jar from the top-level into >> the `lib` directory in the run-time image as we proposed [1]. >> >> Webrev: >> http://cr.openjdk.java.net/~mchung/jdk9/web

Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

2016-11-23 Thread Alan Bateman
On 22/11/2016 21:07, Mandy Chung wrote: This patch moves src.zip and jrt-fs.jar from the top-level into the `lib` directory in the run-time image as we proposed [1]. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ This looks good. A minor point in SystemImage.fi

Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

2016-11-23 Thread Erik Joelsson
Build changes look good. /Erik On 2016-11-22 22:07, Mandy Chung wrote: This patch moves src.zip and jrt-fs.jar from the top-level into the `lib` directory in the run-time image as we proposed [1]. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ thanks Mandy [1] h

Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

2016-11-22 Thread Jim Laskey (Oracle)
+1 > On Nov 22, 2016, at 5:07 PM, Mandy Chung wrote: > > This patch moves src.zip and jrt-fs.jar from the top-level into > the `lib` directory in the run-time image as we proposed [1]. > > Webrev: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ > > thanks > Mandy > [1] >

Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

2016-11-22 Thread Mandy Chung
This patch moves src.zip and jrt-fs.jar from the top-level into the `lib` directory in the run-time image as we proposed [1]. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ thanks Mandy [1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-November/010128.html