On 2016-04-14 19:20, Christian Thalinger wrote:
On Apr 14, 2016, at 6:41 AM, Mikael Vidstedt <mikael.vidst...@oracle.com> wrote:
Please review the following change which removes the "client" VM from the
default JIB build profile
Is there some public documentation about JIB? A quick search only showed a few
JBS bugs that mention JIB.
No, Jib is an Oracle internal tool, just like JPRT. But, just as we need
a JPRT configuration in the open repo, we also need a Jib configuration
in the open repo. Otherwise we cannot build and test open only forests
internally.
/Erik
on windows-x86 and linux-x86:
Bug: https://bugs.openjdk.java.net/browse/JDK-8154209
Webrev (top): http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.01/
Webrev (hotspot):
http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.01/hotspot/webrev/
When not including the client VM, the build system automatically creates a
jvm.cfg which makes -client an alias for -server. At some point in the future
we may choose to output a warning and/or refuse to start up if -client is
specified, but at least for now silently falling back on the -server VM seems
appropriate.
The test/runtime/SharedArchiveFile/DefaultUseWithClient.java test assumes that
CDS is always compiled in and enabled in the -client VM on windows-x86. Since
-client will fall back on -server that is no longer true, so the test needs to
be updated. I added an @ignore and filed the following issue to track fixing
the test:
https://bugs.openjdk.java.net/browse/JDK-8154204
Testing:
In addition to a standard JPRT push job, Christian Tornqvist helped me run the
runtime nightly tests and apart from the above mentioned test all tests were
successful.
Cheers,
Mikael