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.

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.

Reply via email to