| 30 Mar 2025 |
Alexandros Liarokapis | * It's been more than a year since setting this up and was wondering if I can get clean cross-compilation using clang/llvm cross stdenv or similar. | 10:43:51 |
Alexandros Liarokapis | * hi all. I am doing a lot of cross-compiling for thumbv7-none-eabi* targets. I had a custom stdenv setup with the arm multi-arch arm-none-eabi toolchain. That required doing a manual nixpkgs.lib.init instantiation with the /pkgs/stdenv/cross function and custom binutils wrapping. | 10:44:16 |
| 31 Mar 2025 |
Tristan Ross | Recently had the idea for meta.teams (https://github.com/NixOS/nixpkgs/pull/394797) to be able to list teams which owns a package as an alternative to meta.maintainers. | 05:01:05 |
Sandro 🐧 | That kinda already exists https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/openstacksdk/default.nix#L82 | 10:14:55 |
Tristan Ross | In reply to @sandro:supersandro.de That kinda already exists https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/openstacksdk/default.nix#L82 Yeah but you can't check anything aside from members. This provides more information. | 14:05:26 |
| Grimmauld (moving to @grimmauld:grapevine.grimmauld.de) changed their display name from Grimmauld (any/all) to Grimmauld (moving to @grimmauld:grapevine.grimmauld.de). | 14:19:16 |
| Grimmauld (any/all) joined the room. | 15:05:52 |
| 1 Apr 2025 |
| Sandro 🐧 changed their display name from Sandro 🐧 to Sandro 🐧 [c3d2]. | 13:56:19 |
| 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 |