Conversation
SuperSandro2000
left a comment
There was a problem hiding this comment.
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 |
mmilata
left a comment
There was a problem hiding this comment.
Didn't test but Jitsi changes look reasonable.
doronbehar
left a comment
There was a problem hiding this comment.
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
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore 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