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.