| 11 Oct 2023 |
 vcunat | Uh, I made some mistake. I don't think it succeeded. I probably omitted the patch by mistake. | 09:54:30 |
| vcunat | I don't know, I'll probably give it up for the current staging-next-23.05. In case someone wants to experiment, you can get prototype patch for nghttp2 (version without touching generated stuff): https://github.com/vcunat/nghttp2/pull/new/p/backport-cve-2023-44487 | 10:20:33 |
| 14 Oct 2023 |
 p14 | I've just discovered (messing around on staging) that a nix shell .#pkgsLLVM.buildPackages.stdenv.cc gives you a x86_64-unknown-linux-clang can't compile things. The linker fails, unable to find libunwind and friends. The reason is that the logic for adding the appropriate -L flags is actually handled by the bintools setuphook. The packages are named as extraPackages in the clang wrapper: https://github.com/NixOS/nixpkgs/blob/176015fc743583e30581c23bfda2bf848dd4d67d/pkgs/development/compilers/llvm/16/default.nix#L247-L252
This results in NIX_LDFLAGS having a -L/path/to/libunwind (and libcxx, etc), if you're in a stdenv, and not otherwise.
My conclusion is that the compiler only works if you're in a stdenv, but I feel that it should work standalone. Is that reasonable? I think this could be fixed by baking it into the cc-cflags as is done for compiler-rt here. https://github.com/NixOS/nixpkgs/blob/176015fc743583e30581c23bfda2bf848dd4d67d/pkgs/development/compilers/llvm/16/default.nix#L257
| 13:54:38 |
| p14 | Seems someone already beat me to it: https://github.com/NixOS/nixpkgs/issues/201591 / https://github.com/NixOS/nixpkgs/pull/220520 | 14:13:40 |
 trofi | yeah, i think mingw compilers have a similar problem and don't supply enough libraries to compile hello world without manual fiddling | 21:00:38 |
| 15 Oct 2023 |
| vcunat | BTW, I thought the normal way to change C compiler is to use overridden stdenv, here probably llvmPackages.stdenv or clangStdenv. | 05:59:49 |
| vcunat | Unfortunately I don't know a way of doing that in a shell except writing a shell.nix. | 06:00:24 |
| vcunat | * Unfortunately I don't know a way of doing that in a nix shell except writing a shell.nix. | 06:00:34 |
| trofi | Yeah, that matches my understanding. AFAIU `nix shell` does not create enough development environment (like running hooks). | 06:18:21 |
| 16 Oct 2023 |
| p14 | I don't see why it couldn't work though, right, I mean, we're building a compiler wrapper, that compiler wrapper should have enough within it to (trivially) function. Even if you have to make a stdenv in order to have any more interesting dependencies. | 16:49:36 |
| trofi | It could work in theory. I would even argue that cc should work without any wrappers and fancy variables like NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu=1. | 21:00:14 |
| 17 Oct 2023 |
| vcunat | What I wrote was mainly about my experience (over many years). I don't recall details. | 07:45:32 |
| vcunat | It's like, in practice some complex edge cases are difficult. | 07:47:27 |
| 21 Oct 2023 |
|  Dusk joined the room. | 04:54:00 |
| 23 Oct 2023 |
|  Puna joined the room. | 20:52:50 |
| 24 Oct 2023 |
|  chivay joined the room. | 23:10:53 |
| 25 Oct 2023 |
|  Federico Damián Schonborn changed their profile picture. | 00:13:06 |
|  fgaz joined the room. | 07:39:20 |
| 26 Oct 2023 |
|  streets joined the room. | 12:33:22 |
| 27 Oct 2023 |
| Federico Damián Schonborn changed their profile picture. | 01:24:38 |
| 28 Oct 2023 |
 raitobezarius | Hello there, I think this change https://github.com/NixOS/nixpkgs/pull/263446 is very cool and would be curious of the stdenv people opinion | 12:18:19 |
| 30 Oct 2023 |
|  pbsds joined the room. | 21:16:52 |
| pbsds | Would love some opinions on https://github.com/NixOS/nixpkgs/pull/260776/files | 21:33:02 |
| pbsds | * Would love some opinions on https://github.com/NixOS/nixpkgs/pull/260776 | 21:33:10 |
| raitobezarius | I personally like it | 21:33:21 |
| raitobezarius | (but I have treewide fatigue) | 21:33:39 |
| pbsds | it will be a long transition, grepping the build logs from hydra to remove all the new warnings, but merging it should not break anything | 21:34:50 |
| raitobezarius | I mean, it will be much easier to discuss it post 23.11 | 21:35:24 |
| raitobezarius | Everyone is probably focused on the incoming ZHF / 23.11 | 21:35:36 |
 Artturin | In reply to @pederbs:pvv.ntnu.no
Would love some opinions on https://github.com/NixOS/nixpkgs/pull/260776 how did you grep hydra | 21:37:30 |
