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