-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrade to JDK 11+ #648
Comments
Just a thought based upon recent news: Do we use serialization anywhere? If so: |
I don't believe we use serialization because we want to be compatible with other tools that may not be Java. |
OpenJDK supports zip and tar downloads of the JVM for platforms. This makes packaging fairly easy. I suspect this is the direction to go for an easier install. I also see that JavaFX is include in OpenJDK 11, which may be a direction to go for the stand-alone Java applications. |
Launch4j may have an issue with this https://sourceforge.net/p/launch4j/bugs/186/ |
log4j uses multi-release jar files. i handled this in another project with this commit jpschewe/google-calendar-agenda-cleanup@907c43e. |
I checked the other day and openjdk 12 is the current download, there is no download for openjdk 11. Ubuntu 18.04 is currently shipping openjdk 11 and no option for 12 yet. |
SpotBugs doesn't support java 11 yet. I tried running it with Java 11 and got all kinds of warnings about extra null checks that aren't in the code. This is using version 3.1.12 of SpotBugs. |
The main reason for this switch is that the default stylesheet doesn't support XSLT 2.0 like it says it does and saxon errors out. I was originally not using saxon, however when switching to java 11, this broke. See the following issues for details * spotbugs/spotbugs-gradle-plugin#114 * spotbugs/spotbugs#958 Issue #648
I build using openjdk 11 under Linux and then tried running under openjdk 12 under Windows 10. The launch4j executable complained that java 11.0.0 was required. However I was able to launch it from the command line. Things seem to work fairly well, except launching the html files. I got an error that fll-sw.html couldn't be found even though the correct path was specified. Got some warnings about dnsjava
This is being tracked at dnsjava/dnsjava#8 |
Some clues on getting the Windows executable to work with openjdk 12
The base problem is that there is no installer for openjdk, therefore the registry entries don't get set and the fll-sw.exe (launch4j) thinks there is no Java VM installed, even if it's on the path. |
The method I think that to move to Java 11 we should have the JDK bundled with the at least the Windows installer, #399, so that we don't have issues with users needing to figure out how to install OpenJDK. |
Consider using PMD instead of SpotBugs, it claims to support Java 13. |
I have upgraded to spotbugs 2.0.1 with tool version 4.0.0-beta4, however that still shows errors around the try with resources statements. |
Spotbugs has problems with recognizing the bytecode that javac generates for try with resources and thinks there are redundant checks for null. Removing these checks from our setup until this is fixed. Issue #648 spotbugs/spotbugs#756
Spotbugs has problems with this pattern as well under Java 11. Issue #648 spotbugs/spotbugs#811
This simplifies the readability of the code. Issue #648
rewrite `NativeJsTemporal` to support all common `TemporalField`s
JDK 10 is out. Things to consider:
The text was updated successfully, but these errors were encountered: