-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
glibc: 2.35-224 -> 2.37-8 #188492
glibc: 2.35-224 -> 2.37-8 #188492
Conversation
Jobset: https://hydra.nixos.org/jobset/nixpkgs/pr-188492-glibc-2.36 |
I mean where's the fun if everything would've built fine already 🙃 But to be serious: I didn't expect it to fail that early, but the problem appears to be GCC: https://hydra.nixos.org/build/189053872/nixlog/1/tail |
Also we might want to consider applying the patch to re-enable |
@vcunat so I just fixed a few more failing builds, updated glibc to the latest rev from 2.36 and pulled in latest upstream/staging. Let's see what the next Hydra evaluation says, but it seems quite good IMHO. Do you think it makes sense to add |
Yes why not, added. (that also triggered evaluation) |
So, glibc 2.36 missed NixOS 22.11, right? #194208 (comment) |
Yep, but I didn't have the energy last week to finish this anyways 🤷 |
Rebased another time, rustc on arm should be fixed again, let's see how much is actually broken now. |
Failing Hydra build: https://hydra.nixos.org/build/193330218
Failing Hydra build: https://hydra.nixos.org/build/193374680
Failing Hydra build: https://hydra.nixos.org/build/193338640 We don't need the `meson.build` fixes because (1) lxc4 doesn't use meson and (2) it contains some fixes to make sure it works both with newer and older glibc versions. Since we only ship glibc 2.36 from now on, we can skip that.
Failing Hydra build: https://hydra.nixos.org/build/193347368
Failing Hydra build: https://hydra.nixos.org/build/193339001
Failing Hydra build: https://hydra.nixos.org/build/193332774
The functions pidfd_open & pidfd_send_signal are now part of `glibc`, but defined in the header `sys/pidfd.h` which is not included in `bindings.c`. Failing Hydra build: https://hydra.nixos.org/build/193376808
Failing Hydra build: https://hydra.nixos.org/build/193323749
Failing Hydra build: https://hydra.nixos.org/build/193810432
To correctly find the implementation of `pidfd_open` - which was introduced in glibc 2.36 - an additional `#include` is needed. The newly introduced patch adds that. Failing Hydra build: https://hydra.nixos.org/build/214432977
Failing Hydra build: https://hydra.nixos.org/build/214419156 Relevant upstream thread: https://lore.kernel.org/qemu-devel/[email protected]/
Cutoff for merge to 23.05 would be in two weeks: It's not day-precise in practice, but most likely better merge sooner than later (if for 23.05). Low-rebuild breakages can deferred after merging, unless it's some very important package. |
Can't say much for x86_64-linux-specific breakage (because all of these builds are still hanging in the queue), but aarch64-linux looks quite fine I'd say: A lot of stuff is transitively broken because of apipkg (but restarting the build appears to solve this, I just didn't want to manually restart every single affected job manually, see https://hydra.nixos.org/build/214454805). Same for An obvious problem is swift: https://hydra.nixos.org/build/214392116 (error is reproducible on x86_64-linux). I've never used swift, so no idea about that, but I'd say that swift isn't a sufficiently large subsystem to block on that, IMHO the swift maintainers should figure out a solution during zhf (but I'll give them a heads-up now, cc @stephank @Trundle @dduan @trepetti @dtzWill ). So unless I missed something obvious, there don't seem to be any significant regressions, the last one - which is already fixed - is the QEMU build error. For me this is enough to undraft this, but I'll leave it up to you to merge (perhaps we should at least await the x86_64-linux part first). |
I restarted all failed, as aarch64-linux machines tend to have extra free time lately. |
Sounds good enough for |
Thanks. |
The error there is a simple JSON parse error, which is caused by the extra warning output. I'm not yet sure where that |
FYI, this regression will most likely very soon get to master+unstable (and 23.05). |
It's unfortunate, but I don't have the bandwidth to investigate this right now. What time I find is spent on the Swift 5.8 upgrade. Hopefully that magically fixes it, but probably not. Maybe we can do a 23.05 backport later when we have a fix, but until then we'll have a broken Swift on Linux unless someone else can take a look. I agree it's not a blocker. |
#225197 fixes the Linux build of Swift. I did not find a direct cause, but found issues with CMake >=3.25 in general, so suspect that's what fixed it. (I created an upstream PR. They use an older version of CMake in CI, so didn't notice.) |
Description of changes
Note: very early draft, but opening it to track the work and to avoid potentially duplicated efforts.
cc @lovesegfault @vcunat
Also, can anybody create a Hydra jobset for me, please?
Announcements:
https://sourceware.org/pipermail/libc-alpha/2022-August/141193.html
https://sourceware.org/pipermail/libc-alpha/2023-February/145190.html
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes