Re: RFR [9] Modular Source Code

2014-08-13 Thread Alan Bateman
On 13/08/2014 22:29, Mike Duigou wrote: : - modules.xml "Changes to this file will require review by Committers to Project Jigsaw." Will this be true after integration into jdk9/dev repo? There's a section in JEP 200 about modules.xml, it is temporary until there is a module system in place.

Re: RFR [9] Modular Source Code

2014-08-13 Thread Mike Duigou
There's a lot to review here. This is not a complete review but hopefully contributes to our review "coverage". I am focusing on the top project in this set of comments. - --with-output-sync seems like it should be on by default if available. Downside? This could also be split out from the jigs

Re: RFR [9] Modular Source Code

2014-08-13 Thread Chris Hegarty
Thanks for the explanation Erik. I have taken a pass over the changes, and they look ok to me ( I am happy to be listed as a reviewer ). I also did several build and test runs on Solaris, Linux, Max OSX, and Windows. All look good. I am seeing, in some cases, about a 20% reduction in image bu

Re: RFR [9] Modular Source Code

2014-08-13 Thread Erik Joelsson
On 2014-08-13 14:17, Chris Hegarty wrote: Thanks for the explanation Erik. I have taken a pass over the changes, and they look ok to me ( I am happy to be listed as a reviewer ). I also did several build and test runs on Solaris, Linux, Max OSX, and Windows. All look good. I am seeing, in some

Re: RFR [9] Modular Source Code

2014-08-13 Thread Alan Bateman
Just to add to Chris and Erik's mails, I would encourage everyone that pushes to jdk9/dev or the other jdk9 project integration forests to clone and build jigsaw/stage and get familiar with the proposed layout, new build targets, and the very different output emitted during the build. The cha

Re: RFR [9] Modular Source Code

2014-08-13 Thread Erik Joelsson
I should probably write something about the rather extensive changes to the build logic in this patch. As the source gets organized around modules, it made sense to also organize the build more around modules. In this patch, the makefiles have in large parts been split into module specific fil