RFR(S): 8211350: Remove jprt support

Alan Bateman Alan.Bateman at oracle.com
Tue Oct 2 18:28:26 UTC 2018


On 02/10/2018 19:17, Mikael Vidstedt wrote:
> Thanks for the reviews. I’ve reverted the changes related to Helper and “just” adjusted the comments instead.
>
> webrev:  http://cr.openjdk.java.net/~mikael/webrevs/8211350/webrev.01/open/webrev/
> incremental (from webrev.00): http://cr.openjdk.java.net/~mikael/webrevs/8211350/webrev.01.incr/open/webrev/
>
>
> Btw, I notice that "Test not run, NO jmods directory” will be printed twice when jmods is missing - once in Helper::newHelper and once in the methods calling it. In general, the handling of a null return from newHelper could use some clean up, but that is out of scope for this change.
>
At some point I think the test for jlink will need to be cleaned up 
anyway but that is way outside of scope of what you are doing. The 
updated webrev looks okay to me.

-Alan



More information about the build-dev mailing list