RFR: JDK-8066474: Remove the lib/$ARCH directory from Linux and Solaris images

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Fri Nov 18 20:40:58 UTC 2016


On 2016-11-18 16:30, Erik Joelsson wrote:
> Hello,
>
> Please review this change which removes the $ARCH sub directory in the 
> lib directory of the runtime images, which is an outstanding issue 
> from the new runtime images. Most of the changes are in the build, but 
> there are some in hotspot and launcher source. I have verified 
> -testset hotspot and default in JPRT as well as tried to run as many 
> jtreg tests as possible locally. I could only really find two tests 
> that needed to be adjusted.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8066474
>
> Webrev: http://cr.openjdk.java.net/~erikj/8066474/webrev.01

Looks good to me.

If anything, the switch statement in ProcessImpl.java seems superfluous 
now, and you could possibly prune that bit even harder.

Nice to see this go. :)

/Magnus



More information about the build-dev mailing list