!UNVBThoJtlIiVwiDjU:nixos.org

Staging

315 Members
Staging merges | Running staging cycles: https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+is%3Aopen+head%3Astaging-next+head%3Astaging-next-25.05 | Review Reports: https://malob.github.io/nix-review-tools-reports/108 Servers

Load older messages


SenderMessageTime
7 Oct 2025
@niklaskorz:matrix.orgniklaskorzno worries, PR will be there within the hour10:21:49
@vcunat:matrix.orgVladimír ČunátIf I can just merged it, now-ish is a good moment.10:22:01
@niklaskorz:matrix.orgniklaskorzalright, I'm on it10:22:10
@vcunat:matrix.orgVladimír ČunátAnd if it's not too urgent it can be in the next one which will also get into 25.11.10:22:45
@vcunat:matrix.orgVladimír Čunát* Or if it's not too urgent it can be in the next one which will also get into 25.11.10:22:58
@vcunat:matrix.orgVladimír ČunátBut it may take a couple weeks to reach master.10:23:14
@vcunat:matrix.orgVladimír Čunát* That would delay it by couple weeks (to reach master).10:23:49
@vcunat:matrix.orgVladimír ČunátOh no, fails to build on darwin: https://hydra.nixos.org/build/309261595/nixlog/2/tail10:25:50
@vcunat:matrix.orgVladimír ČunátI suppose I'll just revert the update, unless someone comes with a fix fast.10:27:46
@vcunat:matrix.orgVladimír Čunát(I have no bandwidth to dig into this, especially on darwin.)10:28:08
@niklaskorz:matrix.orgniklaskorzI guess if we do realize it's breaking other packages in staging-next, we can still cherry-pick it10:28:48
@niklaskorz:matrix.orgniklaskorz(but as of now I'm not aware of any)10:29:29
@niklaskorz:matrix.orgniklaskorzsounds like a generic clang issue, not darwin specific10:30:12
@niklaskorz:matrix.orgniklaskorz

relevant diff for gnulib in coreutils 9.8:

-  error (exit_failure, errnum, "%s",
+  error (exit_failure, errnum,
          _("unable to record current working directory"));

they've removed the format string parameter in all cases

10:45:16
@niklaskorz:matrix.orgniklaskorzintroduced in: https://github.com/coreutils/gnulib/commit/9537853d31b02493f6f3a0ffb9b22bef1ed7217a reverted in: https://github.com/coreutils/gnulib/commit/9297749090b01720888dceeb5f6dab3d52dcef4010:47:33
@niklaskorz:matrix.orgniklaskorzhttps://lists.gnu.org/r/bug-gnulib/2025-06/msg00325.html10:47:58
@niklaskorz:matrix.orgniklaskorzthe mailing list suggests this was made redundant by a fix in gettext10:48:39
@niklaskorz:matrix.orgniklaskorz

that and:

Do we care about random warnings on non-GNU systems? I thought
that we care about warnings only on GNU systems, because that's
what we develop on and that's where we have some influence on
the compiler (gcc).

meaning upstream's preferred solution would be to just disable this warning

10:49:34
@vcunat:matrix.orgVladimír Čunát So we can patch that by some -Wno-error=fool conditioned in coreutils on stdenv.isClang? 10:54:20
@vcunat:matrix.orgVladimír Čunát * So we can patch that by some -Wno-error=foo conditioned in coreutils on stdenv.isClang? 10:54:37
@niklaskorz:matrix.orgniklaskorzI think so, testing locally now10:57:47
@niklaskorz:matrix.orgniklaskorz(on darwin)10:58:09
@k900:0upti.meK900 Ah shit 11:00:50
@k900:0upti.meK900 Qt 6.10 just dropped 11:00:50
@k900:0upti.meK900 Hold, I guess 11:00:50
@reckenrode:matrix.orgRandy EckenrodeIt’s a nice to have, but it’s only really needed by the Swift update, which is 26.05. The other cc-wrapper one is more important for the default SDK and min version updates.11:01:23
@reckenrode:matrix.orgRandy EckenrodeSo GCC doesn’t have the warnings because of proprietary GNU extensions?11:05:58
@niklaskorz:matrix.orgniklaskorz

basically:

I think the warning-avoiding fix in gettext.h works only for gcc,
because gcc recognizes 'gettext', 'dgettext', 'dcgettext' as built-ins.

11:06:31
@niklaskorz:matrix.orgniklaskorzfollowed by:11:06:48
@niklaskorz:matrix.orgniklaskorz

the more pragmatic way is to just accept the fact that clang
produces more warnings than gcc, and to filter out / triage the warnings
from the compilation log

11:06:54

Show newer messages


Back to Room ListRoom Version: 6