Detect if bootclasspath/classpath is set in module mode
Karen Kinnear
Karen.Kinnear at Sun.COM
Fri Mar 19 10:43:50 PDT 2010
Mandy,
They both look good. Glad you caught the early null termination and
thanks for adding
comments about temporary paths :-)
thanks,
Karen
On Mar 19, 2010, at 1:35 PM, Mandy Chung wrote:
> Alan,
>
> I updated the webrev that renames is_modular_mode to is_module_mode
> in hotspot and also fixes an embarrassing bug of mine in src/share/
> bin/java.c line 793 that writes '\0' in a bad location.
>
> Webrev at:
> http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/jdk.01/
> http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/hotspot.01/
>
> Thanks
> Mandy
>
> Mandy Chung wrote:
>> Karen, Alan,
>>
>> We don't support bootclasspath and classpath in module mode. I
>> added the check in hotspot and java launcher. Can you review this
>> simple fix?
>>
>> Webrev at:
>> http://cr.openjdk.java.net/~mchung/jigsaw/modular-mode-checks/
>>
>> Thanks
>> Mandy
>
More information about the jigsaw-dev
mailing list