-
Notifications
You must be signed in to change notification settings - Fork 233
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 Protege to Java 11 #822
Comments
Relevant issue: #702 (comment) |
Relevant issue: #753 |
Relevant issue: #732 |
Relevant issue: #825 |
Ensure that performance is ok. See #723 |
B.t.w., how do you plan go about the new Java licensing model? Switch to OpenJDK? |
For Java 11? Yes, we'll switch to OpenJDK. We've done this for WebProtege. |
Does Protege work well with OpenJDK? I did not test. |
I need to double check both of these things |
OK, I just tried Protege with OpenJDK on MacOS 10.14.2: |
Awesome @ykazakov! Funny icons. Will have to look into this. |
For Java 8, my understanding is that we can still distribute the JRE without problems. There just won't be anymore public updates for Java 8. |
@woutermont have you checked if the branch referenced in #732 works for Java11? |
@ykazakov my bad, I thought support for Java 11 was supposed to be ready now ... The fork doesn't work either though. |
@woutermont if it were ready, this ticked would be closed ;-) |
@ykazakov just the same error as mentioned in the issues, does that make sense? |
@woutermont from which of the two branches did you build Protege? The osgi.ee error should have been fixed by the newer version of Apache Felix. |
I just tried this branch on mac os 10.14.2: works fine with java 10.0.2 but not with 11.0.2:
|
Probably related to this. |
Ah, damn, again, my bad ... correct branch indeed gives the java.xml.bind error. Similar for the other solution offered in that same issue. |
I think I fixed the errors. Could you give it a try? |
@ykazakov that seems to work! I'll work a bit with it and report any problems ... |
As I wanted to run Protege on my ARM M1 Mac natively, I had to use @ykazakov branch and I can confirm that it builds and runs correctly under the ARM native versio of openjdk@11 (installed using homebrew). openjdk version "11.0.9" 2020-10-20 Thanks ! |
@serasset Nice! I did not know that there is already a native java vor M1. |
Any chance that Protege 5.5 will be upgraded to run under OpenJDK 11 (or later)? This issue is already quite old... |
For security I like to wrap everything in Firejail, including Java. To use Java 11, in addition to the changes mentioned above, for the swrltab plugin I also had to download a CORBA implementation as CORBA was removed from Java 11. Here are the errors on launch regarding swrltab, which were later fixed with the CORBA implementation download:
Without having to setup and re-learn an environment for Java development, I manually downloaded the newer Felix from here, javax.xml.bind jar and dependencies from here, and CORBA implementation jar and dependencies from here. I placed those 6 new jars in Protege-5.5.0/bin/ also renaming the Felix jar to match what was in run.sh. Dependency links are at the right side on the bottom on each of those Maven web pages. In run.sh in addition to javax.xml.bind I added jacorb: Works. I should note that the glassfish-corba-orb CORBA implementation might have better performance than the JacORB implementation so feel free to benchmark if that matters... I just picked the JacORB implementation as it was less jars to download manually. |
Regarding #822 (comment): 9d8c0d4 takes care of this. @matthewhorridge: I can create a pull request if you are interested. |
Closing as Protégé has been using Java 11 since version 5.6.0. Discussions about how/when to move to a higher version should go to #1236. |
No description provided.
The text was updated successfully, but these errors were encountered: