RFR 8058117: Missing jdk.deploy.osx from modules.xml

Mandy Chung mandy.chung at oracle.com
Wed Sep 10 16:01:57 UTC 2014


On 9/10/2014 8:43 AM, Chris Hegarty wrote:
> While looking at the parsing of modules.xml, I came across a potential 
> issue; jdk.deploy.osx is not listed in modules.xml. This was just an 
> oversight in the original generation of modules.xml, as it will only 
> appear on Mac.
>

Looks okay.  Thanks for catching it and the fix.

Mandy

> Note: This is not an issue for the build as jdk.deploy.osx is in 
> modules.list
>
> Trivial patch:
>
> diff --git a/modules.xml b/modules.xml
> --- a/modules.xml
> +++ b/modules.xml
> @@ -1569,6 +1569,12 @@
>      <depend>jdk.crypto.ec</depend>
>    </module>
>    <module>
> + <name>jdk.deploy.osx</name>
> + <depend>java.base</depend>
> + <depend>java.desktop</depend>
> + <depend>java.scripting</depend>
> + </module>
> + <module>
>      <name>jdk.dev</name>
>      <depend>java.base</depend>
>      <depend>java.scripting</depend>
>
> -Chris.



More information about the jigsaw-dev mailing list