Skip to content
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

jide-oss fails to build from source with OpenJDK 10 #30

Open
apoleon opened this issue May 1, 2018 · 5 comments
Open

jide-oss fails to build from source with OpenJDK 10 #30

apoleon opened this issue May 1, 2018 · 5 comments

Comments

@apoleon
Copy link

apoleon commented May 1, 2018

Hello,

the latest version of jide-oss fails to build from source with OpenJDK 10. OpenJDK 9 is end-of-life now.

I'm hereby forwarding Ubuntu bug

https://bugs.launchpad.net/ubuntu/+source/libjide-oss-java/+bug/1766131

[javac] /<>/libjide-oss-java-3.7.2+dfsg/src/com/jidesoft/plaf/LookAndFeelFactory.java:29: error: package com.sun.java.swing.plaf.windows does not exist
[javac] import com.sun.java.swing.plaf.windows.WindowsLookAndFeel;

It seems that it's been broken by this change in the jdk: https://bugs.openjdk.java.net/browse/JDK-8189656 ("The Windows L&F should be moved out from the shared folder") which means that the com.sun.java.swing.plaf.windows classes are not built as part of our jdk10 distribution.

@jidesoft
Copy link
Owner

jidesoft commented May 2, 2018

I didn't even know jide oss is part of the Ubuntu release? We will try to fix it. It is not easy as we used Windows L&F a lot in the code. I am not sure even sure if it is possible to remove all usages.

@apoleon
Copy link
Author

apoleon commented May 2, 2018

jide oss has been included in Debian since 2013 and from there it became part of Ubuntu and other derivatives. It seems that Windows L&F is not gone but was just moved to a different location. It is hopefully just a lot of search&replace.

@jidesoft
Copy link
Owner

jidesoft commented May 2, 2018

I pushed a change. Please try it again. There will still be a few class that I don't know how to solve. For example, public class VsnetWindowsProgressBarUI extends WindowsProgressBarUI.

Affected classes are
VsnetWindowsProgressBarUI.java
WindowsJidePopupMenuUI.java
WindowsRangeSliderUI.java
XertoJideButtonUI.java

Those are Windows specific UI classes but you won't be able to compile on Linux.

@apoleon
Copy link
Author

apoleon commented May 3, 2018

I can confirm that Windows specific classes make the build fail on Linux with OpenJDK 10 at the moment.

It also fails now because the class WindowsGraphicsUtilsPort cannot be found. I probably could remove Windows specific UI classes as a last resort but I still hope there is a better solution.

@jidesoft
Copy link
Owner

jidesoft commented May 3, 2018

I just pushed the file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants