RFR: JDK-8179078: Jib run-test-prebuilt profile missing dependency on bootjdk (original) (raw)

Christian Tornqvist christian.tornqvist at oracle.com
Fri Apr 21 14:48:08 UTC 2017

Hi Erik,

This looks good, I’ve verified that the fix works. Thanks for fixing this!

Thanks, Christian

On Apr 21, 2017, at 10:43 AM, Erik Joelsson <erik.joelsson at oracle.com> wrote:

(resending to include Christian)

On 2017-04-21 16:37, Erik Joelsson wrote: Hello,

When using the run-test-prebuilt Jib profile to run tests, we need to set JTJAVA to point to the boot jdk for jtreg. The configuration to set JTJAVA is there, but I forgot to also add bootjdk as a dependency on that profile. Bug: https://bugs.openjdk.java.net/browse/JDK-8179078 Patch: diff -r 111e2e7d00f4 common/conf/jib-profiles.js --- a/common/conf/jib-profiles.js +++ b/common/conf/jib-profiles.js @@ -584,7 +584,7 @@ var testOnlyProfilesPrebuilt = { "run-test-prebuilt": { src: "src.conf", - dependencies: [ "jtreg", "gnumake", testedProfile + ".jdk", + dependencies: [ "jtreg", "gnumake", "bootjdk", testedProfile + ".jdk", testedProfile + ".test", "src.full" ], workdir: input.get("src.full", "installpath") + "/test",

/Erik

More information about the build-dev mailing list