Review request 8136930: Simplify use of module-system options by custom launchers

[Alexandre (Shura) Iline]

> On Aug 11, 2016, at 1:06 PM, Jonathan Gibbons <jonathan.gibbons at oracle.com> wrote:
> 
> Shura,
> 
> Such is the nature of things, I would expect there are some instances of the old style options inbound from separate changesets under development or in review. I would expect that we will do a separate additional pass over tests before we remove support for the old style options.

Of course.

And also, to be completely sure that the tests using new options everywhere, a test run needs to be performed for all tests and failures need to be looked on. Which we will do once the promotion build is out.

Shura

> 
> -- Jon
> 
> On 08/11/2016 09:16 AM, Alexandre (Shura) Iline wrote:
>>> On Aug 11, 2016, at 7:57 AM, Mandy Chung <mandy.chung at oracle.com> wrote:
>>> 
>>> Tests in jdk, hotspot, langtools, jaxp and nashorn repos have been converted to use the new options when this fix was pushed.
>>> 
>>> Any test changes in jdk9/client or jdk9/hs that are not yet in jdk9/dev would require changes.
>> Just to be absolutely clear, you have scanned the source files for appearances of the options which are to go away, correct? You have searched java and shell files, I assume.
>> 
>> I hope it covers all the cases. If there is anything else we should be able to quickly identify it after running the tests.
>> 
>> Shura
>> 
>> 
>>> Mandy
>>> 
>>>> On Aug 11, 2016, at 7:47 AM, Alexandre (Shura) Iline <alexandre.iline at oracle.com> wrote:
>>>> 
>>>> Hi, Mandy.
>>>> 
>>>> Could you help to identify what tests in the JTreg suite(s) require more work to switch to the new options?
>>>> 
>>>> Or the other way around: which tests have been fixed with this commit?
>>>> 
>>>> Thank you.
>>>> 
>>>> 
>>>>> On Aug 5, 2016, at 1:11 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>>>>> 
>>>>> This patch renames the module-system options to GNU-style as specified
>>>>> in JEP 293 [1] (see below for the new proposed option names).  This
>>>>> addresses the problems discussed in [2] that the launcher will pass
>>>>> the module-system options down to the VM in the form of <option>=<value>.
>>>>> This provides a consistent way to configure the module system and
>>>>> simplify use of module-system options by custom launcher.  This patch
>>>>> also updates several JDK tools including jlink, jmod, jimage, jar,
>>>>> javac, javap, javadoc, javah, jdeps such that the GNU-style
>>>>> module-system options are consistent across all tools.
>>>>> 
>>>>> Webrev:
>>>>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8136930/gnu-options/webrev.00/
>>>>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8136930/gnu-options/webrev-langtools.00/
>>>>> 
>>>>> Harold has posted a separate code review for the hotspot change [3].
>>>>> webrev.00 includes changes in all repos except hotspot.  webrev-langtools.00
>>>>> includes the changes in langtools repo (this may be convenient for
>>>>> those who wants to review langtools change only).
>>>>> 
>>>>> Note that existing -cp and -classpath have no change and continue to
>>>>> be supported.  New long form option `--class-path` and `--help` are
>>>>> added to java, javac, and other tools where applicable.
>>>>> 
>>>>> For transition, all old options except -listmods continue to be
>>>>> supported by the java launcher, javac, javadoc, javap, javah.
>>>>> I propose to remove the old options in two weeks after this patch
>>>>> is promoted (i.e. two promoted builds to go through the transition).
>>>>> 
>>>>> Mandy
>>>>> 
>>>>> [1] http://openjdk.java.net/jeps/293
>>>>> [2] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-June/008079.html
>>>>> [3] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-July/008715.html
>>>>> 
>>>>> Existing Options                    New Options
>>>>> 
>>>>> -addmods                            --add-modules
>>>>> -classpath | -cp | --classpath      -classpath | -cp | --class-path
>>>>> -limitmods                          --limit-modules
>>>>> -listmods                           --list-modules
>>>>> -m                                  --module | -m
>>>>> -modulepath | -mp | --modulepath    --module-path | -p
>>>>> -modulesourcepath                   --module-source-path
>>>>> --plugin-module-path                (no change)
>>>>> -processormodulepath                --processor-module-path
>>>>> -upgrademodulepath                  --upgrade-module-path
>>>>> -XaddExports                        --add-exports
>>>>> -XaddReads                          --add-reads
>>>>> -Xpatch                             --patch-module
>>>>> -bootclasspath                      --boot-class-path | -bootclasspath
>>>>> -processorpath                      --processor-path  | -processor-path
>>>>> -sourcepath                         --source-path | -sourcepath
>>>>> -system                             --system
>>>>> -release                            --release
>