| 1 Apr 2025 |
| Sandro 🐧 changed their display name from Sandro 🐧 [c3d2] to "Sandro [c3d2]". | 13:57:05 |
| Sandro 🐧 changed their display name from "Sandro [c3d2]" to Sandro 🐧 [c3d2]. | 13:57:41 |
| Sandro 🐧 changed their display name from Sandro 🐧 [c3d2] to Sandro 🐧. | 13:58:48 |
| 5 Apr 2025 |
| underpantsgnome removed their display name underpantsgnome. | 15:54:36 |
| underpantsgnome left the room. | 15:59:31 |
| 10 Apr 2025 |
Tristan Ross | So WG14 is looking to make many libc functions use const char* instead of char* after like 40 years. This brings a problem where many things aren't expecting this. There will be a C revision which includes this so we don't have to deal with it immediately. However, one of the WG14 people discussed this with the LLVM libc team which I am a part of. I've found a few things broken via Nix. One of which seems to be makeWrapper, -Werror=write-strings is a good mechanism for catching this. Do we want to enable that flag in a few places so we can start catching this? | 19:04:23 |
Alyssa Ross | I think it depends if we have people who want to be part of fixing upstreams, which will be a lot of tedious work but a nice way to do good. Usually Fedora and Gentoo do most of that but would be great for us to do more. | 19:06:06 |
Tristan Ross | Yeah, that's why I tested things here. | 19:06:23 |
Tristan Ross | openssl and expect both fail in the fixupPhase but key-utils itself actually has a problem. | 19:06:50 |
Tristan Ross | It's likely we won't see the downstream effect of the const char* stuff until a long time from now but working on the problem sooner than later may not be a bad idea. | 19:07:30 |
Tristan Ross | Aaron Ballman is who I've been on contact with for this btw. | 19:08:12 |
Alyssa Ross | Yeah | 19:08:15 |
Alyssa Ross | We could maybe have a low-activity Hydra jobset? | 19:08:24 |
Tristan Ross | Yeah, that may not be a bad idea | 19:08:36 |
Alyssa Ross | Great that you're involved in this so proactively | 19:08:59 |
Tristan Ross | As a quick and dirty way to get started, I just did this:
diff --git a/pkgs/build-support/cc-wrapper/default.nix b/pkgs/build-support/cc-wrapper/default.nix
index 692474d48c42..262594e6dd15 100644
--- a/pkgs/build-support/cc-wrapper/default.nix
+++ b/pkgs/build-support/cc-wrapper/default.nix
@@ -845,6 +845,10 @@ stdenvNoCC.mkDerivation {
substituteAll ${./add-clang-cc-cflags-before.sh} $out/nix-support/add-local-cc-cflags-before.sh
''
+ + ''
+ echo "-Werror=write-strings" >> $out/nix-support/cc-cflags
+ ''
+
##
## Extra custom steps
##
| 19:08:59 |
Alyssa Ross | Yeah I'd just throw that up on a branch and get Hydra on it for now | 19:09:30 |
Alyssa Ross | I'm not sure whether we'd want to enable this for real before compilers start doing so | 19:11:46 |
Alyssa Ross | But getting a head start is good | 19:11:51 |
emily | agreed that preparing for this stuff ahead of time is great :) | 19:12:06 |
emily | I wish we had the resources to have an "experimental" jobset always running that we could throw stuff like this on | 19:12:27 |
emily | rather than dealing with things just-in-time during staging cycles | 19:12:35 |
Alyssa Ross | It's the sort of thing that somebody could probably run on a personal Hydra if they wanted I reckon? We don't need the results particularly fast given the bottleneck is actually going to be manually fixing things one at a time. | 19:13:44 |
emily | I just meant in general, put a bunch of upcoming breaking stuff behind a flag and collectively chip away at it gradually | 19:14:35 |
emily | agreed that it's not necessary for this, it'd just be a nice thing to have | 19:14:53 |
Alyssa Ross | We have pkgsExtraHardening also | 19:15:29 |
Alyssa Ross | Or whatever it's called | 19:15:36 |
Tristan Ross | In reply to @emilazy:matrix.org I wish we had the resources to have an "experimental" jobset always running that we could throw stuff like this on Yeah, I've been thinking of that | 19:20:43 |
Tristan Ross | I likely will be able to deploy my own Hydra locally | 19:22:27 |
Tristan Ross | I just got 2 128 core Ampere chips | 19:22:49 |