-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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: 14.0.2 -> 15.0.1 #107547
openjdk: 14.0.2 -> 15.0.1 #107547
Conversation
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.
Ofborg fails because jdk14 is used as an input here https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/networking/apache-directory-studio/default.nix#L1.
dafc86b
to
5144028
Compare
|
@jerith666 There are 282 packages that will get affected by this:
Unfortunately, this kind of major version updates require a lot of manual testings, especially for jdk, see #89731 . Ideally, we should test each package individually and make sure it launches in runtime, but it's probably a lot of work.. Testing 10% percent of these packages will be persuasive and should help this PR get accepted. |
nix-review output11 packages marked as broken and skipped: 2 packages failed to build: 251 packages built: graalvm8 and jvmci8 fail with manual testingPackages that I tested and that worked fine:
Packages that didn't work and work on master:
Packages that didn't work but also didn't work with master either:
summaryThe only packages I found that didn't work here but did work on master were arduino and jitsi-videobridge. There seems to be an update available for arduino so maybe that will fix it. I'm taking care of the GC flags in jitsi videobridge and opened a bug report upstream at jitsi/jitsi-videobridge#1547. A lot of the affected packages are duplicates, as in multiple derivations used for the same application, for example Kodi and its plugins make up 14 derivations. |
Thanks for the report. So we should add commits that will pin them to jdk14. |
5144028
to
d9d16e2
Compare
@jerith666 if you could please take care of |
I just tried
I get past the splash screen and was able to load and compile an example in both of those builds. Here's the output from the one using jdk 15:
@petabyteboy said "I'm taking care of the GC flags in jitsi videobridge", so I'll wait for that but I'm sure I can figure it out myself if needed. :) |
* update two explicit references to jdk14 to just jdk, which no longer points at jdk8 after NixOS#89731. * patch an explicit -XX:+UseConcMarkSweepGC to -XX:UseG1GC, as the former now throws an error (after having been deprecated since jdk 9)
d9d16e2
to
13e3d2b
Compare
Just force-pushed a new commit that includes a patch for |
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.
Didn't test but Jitsi changes look reasonable.
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.
I too didn't test a thing but all changes look good and I believe the OP has tested what they tested to be functional.
I've got two workdays under my belt with this now (running Eclipse on top of this JDK), and no issues to report. So, removing the WIP label; I think this is ready to merge. |
OK. We can't wait for such major updates for ever. Let's cook this in unstable :). |
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)I am off work until early January, so won't be able to confirm this survives a few workdays until then. :)
continues from #83116. fixes #98784.
cc @edwtjo @shlevy @NeQuissimus @lopsided98 @xfix @petabyteboy @lopsided98 @bachp