Erik:

Looks good to me as well.

Tim

On 02/05/19 00:01, Magnus Ihse Bursie wrote:
On 2019-02-05 01:24, Erik Joelsson wrote:
This patch was a bit premature, but now with a proper fix in Jib, here
is an updated version. It restores the use of the new devkit as well
as changes some more instances of install_path to home_path.

Webrev: http://cr.openjdk.java.net/~erikj/8218177/webrev.02/
Looks good to me.

/Magnus


/Erik


On 2019-01-31 16:23, Erik Joelsson wrote:
With the new devkits I deployed recently (Oracle internal), the file
layout changed as a top level directory inside the tar archive was
introduced. This works in most cases, but there are certain
situations where jib does not handle this well. This causes AOT
testing on Linux to stop working with the run-test-prebuilt make target.

This issue is being fixed in Jib, but to activate the fix a bump in
the format_version is required. This patch does that, as well as
adjusting to conform to the new behavior.

Bug: https://bugs.openjdk.java.net/browse/JDK-8218177

Webrev: http://cr.openjdk.java.net/~erikj/8218177/webrev.01/

(note that the Jib fix is not yet live in production)

/Erik



Reply via email to