Skip to content

Bump nixpkgs to 26.05#15927

Open
xokdvium wants to merge 7 commits into
masterfrom
nixpkgs-26.05
Open

Bump nixpkgs to 26.05#15927
xokdvium wants to merge 7 commits into
masterfrom
nixpkgs-26.05

Conversation

@xokdvium
Copy link
Copy Markdown
Contributor

Motivation

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@xokdvium xokdvium requested a review from edolstra as a code owner May 27, 2026 20:41
@xokdvium
Copy link
Copy Markdown
Contributor Author

@artemist, the unprivileged daemon tests don't work somehow. Could you take a stab at it?

@xokdvium
Copy link
Copy Markdown
Contributor Author

Ah great, link-check is broken.

@xokdvium xokdvium mentioned this pull request May 28, 2026
2 tasks
@artemist
Copy link
Copy Markdown
Member

artemist commented Jun 1, 2026

@artemist, the unprivileged daemon tests don't work somehow. Could you take a stab at it?

Sorry I missed this, I'll look into it now

@artemist
Copy link
Copy Markdown
Member

artemist commented Jun 1, 2026

Seems to be a nixpkgs issue. I've made a PR (NixOS/nixpkgs#526816), but if that isn't merged quickly we can add a workaround to the test:

systemd.services.register-nix-paths.serviceConfig = {
  User = config.nix.daemonUser;
  Group = config.nix.daemonGroup;
};

xokdvium and others added 6 commits June 5, 2026 21:47
Suggested in #15927 (comment)

Co-authored-by: Artemis Tosini <artemis.tosini@obsidian.systems>
Claude made this. I don't know what's going on with print.html and I
don't care.

(cherry picked from commit d2bbe24)
This was the sanest solution I could come up with for settings that
are defined in a base class that ends up being inherited from for different
store types (since store slugs are different for each of those).
@github-actions github-actions Bot added new-cli Relating to the "nix" command store Issues and pull requests concerning the Nix store labels Jun 5, 2026
@xokdvium
Copy link
Copy Markdown
Contributor Author

xokdvium commented Jun 5, 2026

@artemist, applied your fix for now. Also figured out the busted link check.

@xokdvium
Copy link
Copy Markdown
Contributor Author

xokdvium commented Jun 5, 2026

Mingw is completely busted here because sqlite with tcl doesn't build. We probably shouldn't block on this

@xokdvium
Copy link
Copy Markdown
Contributor Author

xokdvium commented Jun 5, 2026

@Ericson2314, @amaanq, please look into the CA test failures with an older daemon:

nix-daemon-compat-tests> warning: the daemon is missing the 'realisation-with-path-not-hash' protocol feature, needed to support content-addressing derivations
nix-daemon-compat-tests> warning: the daemon is missing the 'realisation-with-path-not-hash' protocol feature, needed to support content-addressing derivations
nix-daemon-compat-tests>
nix-daemon-compat-tests>
nix-daemon-compat-tests> terminating due to unexpected unrecoverable internal error: Assertion 'outPath' failed in void nix::main_nix_build(int, char**) at /build/source/src/nix/build/../nix-build/nix-build.cc:729
nix-daemon-compat-tests> Nix crashed. This is a bug. Please report this at https://github.com/NixOS/nix/issues with the following information included:
nix-daemon-compat-tests>
nix-daemon-compat-tests> std::terminate() called without exception
nix-daemon-compat-tests> Stack trace:
nix-daemon-compat-tests>  0# 0x0000000000131EEA in nix-build
nix-daemon-compat-tests>  1# 0x00000000000C539A in /nix/store/chqq8mpmpyfi9kgsngya71akv5xicn03-gcc-15.2.0-lib/lib/libstdc++.so.6
nix-daemon-compat-tests>  2# std::unexpected() in /nix/store/chqq8mpmpyfi9kgsngya71akv5xicn03-gcc-15.2.0-lib/lib/libstdc++.so.6
nix-daemon-compat-tests>  3# 0x00000000000D0DFF in /nix/store/scsdg84rrlx75dyvbxybm5n2kav040k6-nix-util-2.35.0pre/lib/libnixutil.so.2.35.0
nix-daemon-compat-tests>  4# __wrap___assert_fail in nix-build
nix-daemon-compat-tests>  5# 0x00000000000E46C0 in nix-build
nix-daemon-compat-tests>  6# nix::mainWrapped(int, char**) in nix-build
nix-daemon-compat-tests>  7# nix::handleExceptions(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, nix::fun<void ()>) in /nix/store/scsdg84rrlx75dyvbxybm5n2kav040k6-nix-util-2.35.0pre/lib/libnixutil.so.2.35.0
nix-daemon-compat-tests>  8# main in nix-build
nix-daemon-compat-tests>  9# 0x000000000002B285 in /nix/store/57iz36553175g3178pvxjij8z5rcsd4n-glibc-2.42-61/lib/libc.so.6
nix-daemon-compat-tests> 10# __libc_start_main in /nix/store/57iz36553175g3178pvxjij8z5rcsd4n-glibc-2.42-61/lib/libc.so.6
nix-daemon-compat-tests> 11# _start in nix-build
nix-daemon-compat-tests>
nix-daemon-compat-tests> ./post-hook.sh: line 23: 64104 Aborted                    (core dumped) nix-build -o "$TEST_ROOT"/result dependencies.nix --post-build-hook "$pushToStore"
nix-daemon-compat-tests> +++(./post-hook.sh:23) onError
nix-daemon-compat-tests> +++(/build/source/tests/functional/common/functions.sh:241) set +x
nix-daemon-compat-tests> post-hook.sh: test failed at:
nix-daemon-compat-tests>   source in ./post-hook.sh:23
nix-daemon-compat-tests>   main in post-hook.sh:10

@Ericson2314
Copy link
Copy Markdown
Member

OK we're on it. Weird because that the compat logic is supposed to be able to pull the output store path out in that old protocol case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation new-cli Relating to the "nix" command store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants