-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[WIP] JDK9 Migration #3888
[WIP] JDK9 Migration #3888
Conversation
# Conflicts: # build.gradle
- brute force attempt at making JabRef runnable with Java 9 - simply removed all the things Java 9 complains about (without fix yet) Split Packages: - LibreOffice - ArchUnit Invalid Module Name: - Latex2Unicode Internal APIs: - only JFX
InputMethodSupport.inputMethodEventComposed(t, e.getCommittedCharacterCount()), | ||
t.substring(0, e.getCommittedCharacterCount()), | ||
insertionIndex); | ||
// TODO: temporarily removed as this is internal API |
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.
Fortunately with the migration to javafx in the maintable-beta branch #3621 this class is no longer needed. So you can leave it like that
@@ -168,7 +185,8 @@ dependencies { | |||
testCompile 'org.reflections:reflections:0.9.11' | |||
testCompile 'org.xmlunit:xmlunit-core:2.5.1' | |||
testCompile 'org.xmlunit:xmlunit-matchers:2.5.1' | |||
testCompile 'com.tngtech.archunit:archunit-junit:0.5.0' | |||
// TODO: has a split package between archunit and archunit-junit | |||
// testCompile 'com.tngtech.archunit:archunit-junit:0.5.0' |
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.
I just saw that they already made their lib compataible with java9 (requires snapshot version=
TNG/ArchUnit#47
This includes #3421
This compiles and runs with Oracle JDK9.