fixes for java.ext.dirs on Windows (more for processing#4623)

benfry · benfry · commit a3f66ab389f0 · 2016-08-19T15:39:02.000-04:00
diff --git a/build/windows/config-cmd.xml b/build/windows/config-cmd.xml
@@ -30,7 +30,7 @@
   <jre>
     <path>java</path>
     <!-- Deal with jokers who install JOGL, ANTLR, etc system-wide -->
-    <opt>-Djava.ext.dirs="%EXEDIR%/java/lib/ext"</opt>
+    <opt>-Djava.ext.dirs="%EXEDIR%\\java\\lib\\ext"</opt>
     <!-- Prevent a user-installed JNA from conflicting with our version.
          https://github.com/processing/processing/issues/2239 -->
     <opt>-Djna.nosys=true</opt>
diff --git a/build/windows/config.xml b/build/windows/config.xml
@@ -27,7 +27,7 @@
   <jre>
     <path>java</path>
     <!-- Deal with jokers who install JOGL, ANTLR, etc system-wide -->
-    <opt>-Djava.ext.dirs="%EXEDIR%/java/lib/ext"</opt>
+    <opt>-Djava.ext.dirs="%EXEDIR%\\java\\lib\\ext"</opt>
     <!-- https://github.com/processing/processing/issues/2239 -->
     <opt>-Djna.nosys=true</opt>
     <!-- Because nosys is set to true, the DLL in the current working 
diff --git a/java/src/processing/mode/java/JavaBuild.java b/java/src/processing/mode/java/JavaBuild.java
@@ -960,7 +960,7 @@ protected boolean exportApplication(File destFolder,
                           PApplet.javaVersionName +
                           ".jdk/Contents/Home/jre/lib/ext");
       } else if (exportPlatform == PConstants.WINDOWS) {
-        runOptions.append("-Djava.ext.dirs=\"%EXEDIR%/java/lib/ext\"");
+        runOptions.append("-Djava.ext.dirs=\"%EXEDIR%\\java\\lib\\ext\"");
       } else if (exportPlatform == PConstants.LINUX) {
         runOptions.append("-Djava.ext.dirs=\"$APPDIR/java/lib/ext\"");
       }

Original file line number	Diff line number	Diff line change
`@@ -960,7 +960,7 @@ protected boolean exportApplication(File destFolder,`
`960`	`960`	`PApplet.javaVersionName +`
`961`	`961`	`".jdk/Contents/Home/jre/lib/ext");`
`962`	`962`	`} else if (exportPlatform == PConstants.WINDOWS) {`
`963`		`- runOptions.append("-Djava.ext.dirs=\"%EXEDIR%/java/lib/ext\"");`
	`963`	`+ runOptions.append("-Djava.ext.dirs=\"%EXEDIR%\\java\\lib\\ext\"");`
`964`	`964`	`} else if (exportPlatform == PConstants.LINUX) {`
`965`	`965`	`runOptions.append("-Djava.ext.dirs=\"$APPDIR/java/lib/ext\"");`
`966`	`966`	`}`