-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
When i add JMetro lib to project - application return me exceptions. #27
Comments
Hi again Pavel, It''s probably because you're not using a compatible Java version. JMetro is meant for Java 8 and below only. You mentioned also using java 10 (didn't really understand exactly which java version you're using)? Skins have changed place in JavaFX 9 and above, so a Class not found exception would happen in this case. Cheers, |
Hi again @dukke, Default i run with Java 10, but i create cmd file with path to Java 8 and run with Java 8, but, application also crash. It maybe because i have JDK 10 and JDK 8. But compile with JDK8. I don't know, good luck with fixing it :) |
What I'm saying is that this probably isn't a JMetro issue but rather an issue with the fact you're using a version of Java greater or equal than Java 9, in this case you're using Java 10. You need to compile and run with Java 8 or below. The skins have changed packages between Java 8 and Java 10, so you'll get this Class not found exception. This will happen with JMetro or any JavaFX library that is made for Java 8 and makes use of skins in "com.sun.javafx.." package. In the future I'll most probably migrate JMetro to Java 11. Right now, unfortunately, I don't have the resources (time, etc) to maintain two separate versions of JMetro for java 8 and for java9+. Java 8 is still by far the most used version of Java, so for the recent future I plan to stick with it. I'll close this issue for now as won't fix. The problem is probably the fact you're using two versions of java at the same time. Try to use Java 8 only and tell me how it goes. Thanks Pavel :) |
No problem @dukke :) Thanks for attention! 👍 |
Hi Pavel,
Did you manage to make it run?
Cheers,
…On Wed, Dec 26, 2018 at 8:21 AM Pavel Erokhin ***@***.***> wrote:
No problem @dukke <https://github.com/dukke> :) Thanks for attention! 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AClQHzCyDgg3gQxfrZ6rxcRrMbAmH_EUks5u8zGYgaJpZM4ZhJU2>
.
--
Pedro Duque Vieira - https://www.pixelduke.com
|
@dukke not, currently my application crash if i add your library. But without library also work :D |
Did you try only to use only Java 8?
That class that you're getting Class not found exception has moved to
another place in Java 9 and above. So the problem is probably that when
running, the Class loader is looking for that class and can't find it,
since in Java9+ it doesn't exist in that package.
…On Wed, Dec 26, 2018 at 6:45 PM Pavel Erokhin ***@***.***> wrote:
@dukke <https://github.com/dukke> not, currently my application crash if
i add your library. But without library also work :D
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AClQH9tkPeScwrHFDakfJw4Ow6PrpPxrks5u88PhgaJpZM4ZhJU2>
.
--
Pedro Duque Vieira - https://www.pixelduke.com
|
@dukke i compile project with Java 8, because TornadoFx support only Java 8 as your library. I run with Java 8 and 10. See video please:) https://www.youtube.com/watch?v=41iTYfNcz9c&feature=youtu.be |
The error I see with java 8 is an NPE.
Can you provide more information about that exception?
…On Wed, Dec 26, 2018 at 7:47 PM Pavel Erokhin ***@***.***> wrote:
@dukke <https://github.com/dukke> i compile project with Java 8, because
TornadoFx support only Java 8 as your library. I run with Java 8 and 10.
See video please:)
https://www.youtube.com/watch?v=41iTYfNcz9c&feature=youtu.be
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AClQHyENYfR8pd-5ko9Ha4_E-yAjj4cJks5u89I_gaJpZM4ZhJU2>
.
--
Pedro Duque Vieira - https://www.pixelduke.com
|
@dukke yes.
The video shows that there are a lot of such exceptions; this is one of them. |
Describe the bug
Steps to add library to project
Materials
Additional information
Environment information
The text was updated successfully, but these errors were encountered: