> On Jan 3, 2017, at 2:45 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
>
>
>>
>>
>> 154 // jdeps --module-path <modulepath> -m root paths
>> 155 String cmd = String.format("jdeps --module-path %s --add-modules
>> %s %s%n",
>> 156 MODS_DIR, roots.stream().collect(Collectors.joining(",")),
>> 157 Arrays.toString(paths));
>>
>> Is the use of Arrays.toString correct here?
>>
>
> Hmm.. why the test is passing with “[]” in the command line? I will check
> that out.
It was correct since this string is used only for printing to the output. The
jdeps command is built programmatically in the test.
> In any case, no need to modify this file. I have reverted it.
>
Updated webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8172212/webrev.01/
Mandy
