Closed icyboxs closed 1 year ago
@icyboxs Thanks for testing jmb, i will add this to my schedule to work on it by the next month hopefully.
Any update? I'm on java 19, I updated the gradle wrapper to use 7.6 and when building it isn't finding the javafx lib. I'm pretty sure you have to declare this in the gradle build since jfx isn't included with the jdk anymore, but it's not there.
Any update? I'm on java 19, I updated the gradle wrapper to use 7.6 and when building it isn't finding the javafx lib. I'm pretty sure you have to declare this in the gradle build since jfx isn't included with the jdk anymore, but it's not there.
Hello @8Keep thanks for testing, yeah, i am going to setup jfx17 with openjdk soon, stay tuned !
I made some local changes, maybe you can build on them:
https://github.com/8Keep/jMonkeyBuilder/commit/0b88731af718588f6bd5d5d7f049acbbaa099866
1) deleted the build dir since it's already in gitignore and don't want old compiled files in the repo 2) Fixed some localization things (unfinished because I didn't feel like typing out all the english words, but the app didn't run with an equivalent en_us properties file 3) Use newest gradle version 4) Use javafx as it's supposed to be in gradle 5) Some fixes to random stuff in the editor, some crappy things that were there and blocked using a newer jfx version
The app now builds with java 19/ jfx 19. It doesn't run however. The problem is that the multithreading in the program is messed up and tries to initialize the jfx thread multiple times. I was trying to fix it but didn't get there.
Fantastic, yeah i agree the multithreading is a little outdated and creepy, but its worth trying though, that saying I may introduce a github discussion here on this repo to fix this issue completely, resolve those runtime errors and update the editor to jfx19/jdk-19, so stay tuned.
If we managed to draw a UML diagram and understand the delegation pattern of the editor, we may fix this error...
Caused by: java.lang.IllegalArgumentException: InputStream cannot be null at java.xml/javax.xml.parsers.DocumentBuilder.parse(DocumentBuilder.java:118) at com.ss.rlib.data.AbstractStreamDocument.parse(AbstractStreamDocument.java:92) at com.ss.editor.config.Config.<clinit>(Config.java:84) ... 16 more
The error points to config.java
@NotNull private static final String CONFIG_RESOURCE_PATH = "/config.xml";
final VarTable vars = new DocumentConfig(EditorUtil.getInputStream(CONFIG_RESOURCE_PATH)).parse();
This value is indeed null.
But you can also see from the diagram that the config.xml file does exist and I don't know why I can't get this file
@icyboxs If you have problem loading this file, then just replace this: https://github.com/jMonkeyBuilder/jMonkeyBuilder/blob/79e09786ad2eb17ef6f462c91e88134bd669cb5a/src/main/java/com/ss/editor/config/Config.java#LL85C1-L89C66
With:
DEV_DEBUG = false;
DEV_DEBUG_JFX = false;
ENABLE_PBR = true;
Until, I investigate into this to see, I think this file is a trash.
@8Keep @icyboxs I have tried jfx-19 with openjdk-17 and with a lot of changes (because the editor uses very old jfx libraries that are not built for new jfx, so very old com.sun
packages are used), so there are a lot of errors, I am giving up on this issue, until I find time to maintain the other old libraries, instead, I will create a building manual using java 8.
build.gradle file
Let me know if you need more information