eessppeelllloo/processing4
1
0
Fork 0
mirror of https://github.com/processing/processing4.git synced 2026-02-02 21:29:17 +01:00
Code Issues Packages Projects Releases Wiki Activity

working on JavaFX on Windows, more runtime problems, needing exports

Browse Source
This commit is contained in:
Ben Fry
2021-06-19 13:20:35 -04:00
parent 687291ab46
commit 8e590b6232
3 changed files with 23 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
java/src/processing/mode/java/JavaBuild.java
View File

@@ -440,6 +440,13 @@ public class JavaBuild {
}
/** Returns the dummy "module" path so that JavaFX doesn't complain. */
public String getModulePath() {
// Just set this to the main core/library directory to pick up JavaFX
return mode.getCoreLibrary().getLibraryPath();
}
/** Return the java.library.path for this sketch (for all the native DLLs etc). */
public String getJavaLibraryPath() {
return javaLibraryPath;

14
java/src/processing/mode/java/runner/Runner.java
View File

@@ -373,6 +373,20 @@ public class Runner implements MessageConsumer {
params.append(javaLibraryPathParam);
// TODO this should only happen with sketches using the JavaFX library
// https://github.com/processing/processing4/issues/209
params.append("--module-path");
params.append(build.getModulePath());
params.append("--add-modules");
//params.append("javafx.base,javafx.controls,javafx.fxml,javafx.graphics,javafx.media,javafx.swing,javafx.web");
params.append("javafx.base,javafx.graphics,javafx.swing");
// TODO Presumably, we need to move away from com.sun.* classes?
// https://github.com/processing/processing4/issues/208
params.append("--add-exports");
params.append("javafx.graphics/com.sun.javafx.geom=ALL-UNNAMED");
params.append("--add-exports");
params.append("javafx.graphics/com.sun.glass.ui=ALL-UNNAMED");
params.append("-cp");
params.append(build.getClassPath());