Review Request: 8173381: osName/osArch/osVersion is missing in ModuleDescriptor created by SystemModules

Alan Bateman Alan.Bateman at oracle.com
Sun Jan 29 21:56:42 UTC 2017


On 28/01/2017 20:50, Mandy Chung wrote:

> :
> I agree that would be cleaner and made the change. That’s what I started with but that’d leave ModuleTarget attribute in the class file.
>
>> For the secret option for testing then I assume it should be "retainModuleTarget" rather than "retainTargetPlatform". There are a couple of places that use the old name in method names and maybe we should rename those too. One other rename is the plugin has "PackagesAttribute" when it should be "ModulePackages" attribute.
> I fixed up a few places in ModuleInfoExtender as well.
>
The updated webrev looks okay and I think okay to keep the ModuleTarget 
attribute with empty values.

A minor comment on UserModuleTest.java is that 
Object.class.getModule().getDescriptor() would be a simpler way to get 
the descriptor for the base module.

-Alan




More information about the jigsaw-dev mailing list