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.joels...@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 >> JT_JAVA to point to the boot jdk for jtreg. The configuration to set JT_JAVA >> is there, but I forgot to also add boot_jdk 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", "boot_jdk", testedProfile + >> ".jdk", >> testedProfile + ".test", "src.full" >> ], >> work_dir: input.get("src.full", "install_path") + "/test", >> >> >> /Erik >> >
