On 9/12/18 2:16 PM, mark.reinh...@oracle.com wrote:
To make this trivial enhancement I first had to fix some broken tests,
so there are two issues and two webrevs here.

8210669: Some launcher tests assume a pre-JDK 9 run-time image layout

   Two launcher tests, ExecutionEnvironment.java and Test7029048.java (in
   test/jdk/tools/launcher), still assume the old image layout in which a
   HotSpot shared library (libjava.so) is found in $JDK/lib/$ARCH/$VMTYPE
   on Linux and Solaris.  The intermediate $ARCH directory was removed in
   JDK 9, as part of JEP 220 (Modular Run-Time Images) [1].  Since that
   time these tests have been succeeding anyway, because they don’t fail
   when the requested VM is not found.

   The tests should be fixed to fail when the requested VM is not found,
   and to look for VM shared libraries in the proper location.

   JBS: https://bugs.openjdk.java.net/browse/JDK-8210669
   Webrev: http://cr.openjdk.java.net/~mr/rev/8210669/

This looks okay.
8210670: Accept double-dash VM-name options at launch time

   Per JEP 293 (Guidelines for JDK Command-Line Tool Options) [2], enhance
   the Java launcher so that `--server` is accepted to select the server VM,
   in addition to `-server`, and likewise for any other VMs listed in the
   $JDK/lib/jvm.cfg file.

   JBS: https://bugs.openjdk.java.net/browse/JDK-8210670
   Webrev: http://cr.openjdk.java.net/~mr/rev/8210670/

jvm.cfg allows to specify an alias.  Have you considered leveraging jvm.cfg alias mechanism?

--server KNOWN ALIASED_TO -server
--client KNOWN ALIASED_TO -client

That'd invoke change in make/Copy-java.base.gmk as well as jvm.cfg to require the option
to start with `--` whereas the alias starts with `-`.

Mandy

Reply via email to