-
Notifications
You must be signed in to change notification settings - Fork 68
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
Building MySlither #2
Comments
Hm, I really should add a build-script. I was unsing an older version of Java-WebSocket, where also I had no slf4j dependencies. There seems to be two issues with darcula:
|
In the mean time, this fork seems to fix some things (haven't tried it though): |
I added a build.gradle so I could build/run an uberjar (vs. running in the memory hog eclipse) |
I fixed these issues on master and updated to protocol version 11. |
Thank you for the darcula fix. I'm going to stick to branch dev for now because EaterBot is my favorite feature. (I'm using it on my server.) |
I built MySlither / dev branch and had three issues.
Java-WebSocket-1.4.0 required changes to the MySlitherWebSocketClient
-import org.java_websocket.drafts.Draft_17;
+import org.java_websocket.drafts.Draft_6455;
-super(serverUri, new Draft_17(), HEADER);
+super(serverUri, new Draft_6455(), HEADER);
Java-WebSocket also required slf4j-api and slf4j-simple
I couldn't get darcula to work. Maybe there's a font not on my Ubuntu system?
Exception in thread "main" java.lang.ExceptionInInitializerError
at com.bulenkov.iconloader.IconLoader.(IconLoader.java:53)
at com.bulenkov.darcula.DarculaLaf.getDefaults(DarculaLaf.java:97)
at javax.swing.UIManager.setLookAndFeel(UIManager.java:539)
at javax.swing.UIManager.setLookAndFeel(UIManager.java:583)
at de.mat2095.my_slither.Main.main(Main.java:11)
Caused by: java.lang.NullPointerException
at com.bulenkov.iconloader.util.UIUtil.initSystemFontData(UIUtil.java:254)
at com.bulenkov.iconloader.util.JBUI.calculateScaleFactor(JBUI.java:47)
at com.bulenkov.iconloader.util.JBUI.(JBUI.java:33)
... 5 more
After that, it looks really good!
The text was updated successfully, but these errors were encountered: