-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
openjfx11: init at 11.0.3+1 #63574
openjfx11: init at 11.0.3+1 #63574
Conversation
really good timing, I needed this to try out bisq about 10 seconds ago. testing it now. |
getting
|
That's what I feared. Let me check. |
Nothing. @jb55 can you fix the checksum and then show me EDIT: that's for |
@abbradar sure give me a sec, I'm running into further build issues so I'm going to update my unstable branch to rule out any weirdness |
I'm getting this on unstable:
does this require stuff from master? It doesn't look like it... |
sigh Okay, I think I know what happens. Give me several minutes. |
1159b10
to
b6f2b32
Compare
Fixed. Fun fact - you need to build close-to-full JavaFX extra time to get all downloaded dependencies. |
my brain isn't parsing this, what do you mean? |
I build JavaFX two times - the first one (thankfully without GStreamer and WebKit but still a full build) to get all dependencies into a fixed-output derivation as a byproduct. |
oh that sounds fun. I got it to build! thanks! |
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.
Briefly skimmed the commit, but was able to test it on an application and it appears to be working!
Hmm this seems to be causing an evaluation failure on darwin?
|
Yeah - non-nixos linux x86_64, |
@risicle I've fixed Darwin problems. No idea what happens with your build though. I've tested in against master now, it builds for me on x86_64 NixOS. Most curious is that it's in OpenJDK, not in OpenJFX. Why would it try to download anything? |
Running it again and it appears to be working this time 🤷♂️ (Edit: yes, |
Very strange, it's not like I changed anything of significance apart from disabling Darwin. Anyway, thanks for testing! I'll merge this in several days if there is no better ideas about closure size.. |
Updates jdk dependency from 8 to 11. Clojure 1.10.0 added support for jdk11, and was released with a new develper tool: REBL (https://github.com/cognitect-labs/REBL-distro). REBL depends on javafx, currently only supported on Nix by jdk11 (see NixOS#63574)
Motivation for this change
JavaFX (modern Java GUI framework) support in our GUI OpenJDK build. This allows for running JavaFX applications on NixOS. I don't know if any way to split it from OpenJDK and install it separately (perhaps via a symlink farm) exists; this is a documented way to integrate OpenJFX.
Before:
After:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)OpenJFX 8 is quite a bit different so it's subject to future work.