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
openjdk11: 11.0.8+10 -> 11.0.8-ga #95123
Conversation
15943bb
to
f7846ae
Compare
otherwise the build of openjfx 14 fails with a disallowed reference to jdk 11, somewhat mysteriously. fixes NixOS#95117
f7846ae
to
dc3f7f0
Compare
Hi, can this PR be reviewed/merged please? I have the same problem for the last few days:
I understand this PR would fix it? |
What's missing for this to be merged please? |
I guess I changed it from "-ga" to "+10" since thats the versioning the jdk is normally versioned. If there's a workaround for having the build number instead of "ga" I'd prefer that. I don't know what's the exact problem with the derivation. /CC @edwtjo |
I agree it would be nice if we understood this better. But, I'm not sure it needs to hold up the merge either. I think one next step to understanding it is to see if we can find out how the build against |
Currently taking a look at it btw :) Would merge after that unless I find something. |
I figured out the source of the problem: The output path of openjdk11_headless is used in a sed command in |
@petabyteboy do you want to fix this here or shall we merge? |
I have a fix that I am testing right now. I think we can close this and I will open another PR with a proper fix in a moment. |
Motivation for this change
Fixes #95117 ... which I don't really know how to square with #94029 (comment), but it's definitely true.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)