-
Notifications
You must be signed in to change notification settings - Fork 62
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
issue-36: Create branch 11.0.0 #39
Conversation
jperedadnr
commented
Oct 6, 2018
- Source/Target compatibility bumped to 1.9
- Core can be used as dependency on any project with Java 9, 10, 11
- Sampler runs standalone in JDK: 9, 10, 11
- Added Launcher class to Sampler to allow running via shadow jar
- Remove private API
- Source/Target compatibility bumped to 1.9 - Core can be used as dependency on any project with Java 9, 10, 11 - Sampler runs standalone in JDK: 9, 10, 11 - Added Launcher class to Sampler to allow running via shadow jar - Remove private API
Excellent work as usual Jose. |
dependencies { | ||
compile "org.openjfx:javafx-graphics:11:win" | ||
compile "org.openjfx:javafx-graphics:11:linux" | ||
compile "org.openjfx:javafx-graphics:11:mac" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is adding all 3 platform a requirement for the fat jar?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this adds the native libs of all three platforms to the fat jar.
If you don't do this, when you create the jar you will have only the current platform, so the jar won't be cross-platform
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for clarifying.
build.gradle
Outdated
repositories { | ||
jcenter() | ||
} | ||
|
||
dependencies { | ||
compile "org.openjfx:javafx-base:11:${platform}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can get rid of the curly braces surrounding platform