 | Nixpkgs Stdenv | 229 Members |
| 74 Servers |
| Nixpkgs Stdenv | 229 Members |
| 74 Servers |
| Sender | Message | Time |
|---|---|---|
| 24 Nov 2024 | ||
 Tristan Ross | emily John Ericson Philip Taron (UTC-8) Randy Eckenrode & Artturin: have you guys seen this post (https://discourse.nixos.org/t/community-team-updates/56458) should we do a meeting to discuss things to add for a reply to the post or do we just want to message in here? | 02:36:45 |
 emily | p14: uh-oh. | 03:28:29 |
| emily | In reply to @rosscomputerguy:matrix.org(I'm going to work on a reply for the Darwin team. not sure what there is to say re: stdenv for the most part) | 03:34:52 |
| emily | In reply to @rosscomputerguy:matrix.org* (I'm going to work on a reply for the Darwin team (w/ Randy). not sure what there is to say re: stdenv for the most part) | 03:35:20 |
| Tristan Ross | In reply to @emilazy:matrix.orgGotcha, yeah that's why I was asking before making a reply. We probably should discuss what to add for it. | 03:36:08 |
 Randy Eckenrode | In reply to @emilazy:matrix.orgThat error drives me crazy, especially when the extra arguments are out of your control. | 06:17:30 |
| emily | I think this might be a regression from the patch → wrapper change | 06:18:31 |
 p14 | In reply to @emilazy:matrix.orgHow are you hitting this? Is there any other context I have missed other than your message? | 11:44:37 |
| emily | irods on staging | 11:44:55 |
| emily | no context, I'm just wondering if this is our first unexpected consequence of the patch :) | 11:45:08 |
| p14 | I don’t understand how that can happen unless it is ending up as a link argument somehow and this is a problem. | 11:45:39 |
| emily | it was during an autoconf thing I think. I forget already | 11:47:30 |
| emily | it's probably the package being silly | 11:47:48 |
| p14 | What is the minimal rebuild way of observing this issue? Also, are there logs somewhere? | 11:49:36 |
| emily | dug it up https://gist.github.com/emilazy/5453176c2e317b5d104b83bdefc910ae | 11:52:57 |
| emily | nix build github:NixOS/nixpkgs/staging#irods reproduces | 11:54:09 |
| emily | but of course has a lot of rebuilds | 11:54:13 |
| emily | it's nanodbc.override { stdenv = llvmPackages_13.libcxxStdenv; } I guess | 11:57:02 |
| p14 | In reply to @emilazy:matrix.org Judging from the log, it's being passed to the linker, and the linker is rejecting it because of -Werror + unused commandline argument. I think in addition to https://github.com/NixOS/nixpkgs/pull/356189, maybe we should have cc-cflags-compile, which just sends flags to the compiler invocation and not the linker? | 14:47:18 |
| p14 | In reply to @emilazy:matrix.org* Judging from the log, it's being passed to the linker, and the linker (link driver, i.e. clang) is rejecting it because of -Werror + unused commandline argument. I think in addition to https://github.com/NixOS/nixpkgs/pull/356189, maybe we should have cc-cflags-compile, which just sends flags to the compiler invocation and not the linker? | 14:48:05 |
| p14 | Otherwise, -Wno-error=no-unused-command-line-argument or --start-no-unused-arguments with --end-no-unused-arguments, but this was only introduced in clang 14: 50ec1306d060e46e0d53c9f5d8a052e1b0d10d3b | 14:50:10 |
 @paparodeo:matrix.org | it seems like a clang warning to me. clang hello.o -nostdlibinc generates a warning in unwrapped clang-13 and in the system clang on ubuntu (clang-14). seems like the package is just broken or expects to use a later clang | 15:18:44 |
| p14 | Yes, when you do clang hello.o this is a link driver invocation (clang invokes the linker under the hood). The problem is then that the compiler is not being invoked; and it's the compiler which would consume the nostdlibinc flag. So the flag remains unused. | 15:20:48 |
| p14 | The problem is that we put nostdlibinc into the cc-cflags, and these flags go to both the linker and the compiler, with no way in nix currently to say that a flag should only go to the compiler invocation only. | 15:21:23 |
| @paparodeo:matrix.org | I see -- clang should consume unused compiler args if it ends up being a link only step, and it seems to on clang-19, but for whatever reason it is not for clang-13 / clang-14. | 15:39:37 |
| p14 | Are you sure it's a compiler version difference and not a patched/unpatched difference? https://github.com/NixOS/nixpkgs/pull/356162/files#diff-7e3eb59be96c7378cf6880e19a4a800591680455a262eb5a5a083917ea67f8edL10 | 15:45:26 |
| p14 | The arg is claimed there. | 15:45:47 |
| p14 | I just tried with clang 13 on nixos-unstable and don't see the warning behaviour (in wrapped clang). | 15:47:20 |
| p14 | Not sure I understand why wrapping would make the warning go away | 15:47:36 |
| @paparodeo:matrix.org | I'm on staging | 15:47:40 |