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
openjdk: 10.0.2 -> 11.0.1 #48326
openjdk: 10.0.2 -> 11.0.1 #48326
Conversation
We'll need a bootstrap jdk for arm before we can build openjdk for that architecture. |
Squashed some of my WIP comments ... I think I need someone who has access to darwin to make that update, though? |
@@ -8098,10 +8095,10 @@ with pkgs; | |||
inherit (darwin) cctools; | |||
inherit (darwin.apple_sdk.frameworks) CoreFoundation CoreServices Foundation; | |||
}; | |||
bazel_jdk10 = callPackage ../development/tools/build-managers/bazel { | |||
bazel_jdk11 = callPackage ../development/tools/build-managers/bazel { |
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.
👍
1ba17e9
to
1d35fa5
Compare
Okay, I've tested that eclipse runs with jdk11 locally. I think once someone can confirm that the darwin jdk11 is basically working, this is ready to merge! |
Tested this to use Bazel with jdk11 on darwin, failed:
It seems like it's using the Linux derivation here:
They're Linux shared libraries:
As opposed to Darwin shared libraries:
|
* the jre is no longer an official part of the jdk (jmod is recommended as a replacement when needing to create smaller runtime images) * darwin continues to use zulu from azul * apps that used 10 now use 11 (eclipse, bazel, josm)
Oh geeze -- facepalm! I just followed the Fixed in 1bd7b98, please try again! |
All good, |
Awesome, thank you! Removed "WIP" from title, ready to merge now! :) |
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.
LGTM
Major TODOs:
jmod
get them buildingneed bootstrap jdk for ARMupdate oraclejdk to 11 tooThings done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)