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 hotspot-dev
mailing list