Skip to content

nixVersions.nix_2_33: init#471463

Merged
Ericson2314 merged 1 commit intoNixOS:masterfrom
philiptaron:nixVersions.nix_2_33
Jan 16, 2026
Merged

nixVersions.nix_2_33: init#471463
Ericson2314 merged 1 commit intoNixOS:masterfrom
philiptaron:nixVersions.nix_2_33

Conversation

@philiptaron
Copy link
Contributor

@philiptaron philiptaron commented Dec 16, 2025

See https://discourse.nixos.org/t/nix-2-33-0-released/73109

Marking as draft until I review the upstream Nix packaging changes around S3.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Dec 16, 2025
@philiptaron philiptaron marked this pull request as ready for review December 17, 2025 01:29
@philiptaron
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 471463 --additional-package nixVersions.latest.tests
Commit: 04c4952f7c082e7d34986e848b686c9dd1da4e32


x86_64-linux

✅ 14 packages built:
  • nixVersions.latest (nixVersions.nix_2_33)
  • nixVersions.latest.dev (nixVersions.nix_2_33.dev)
  • nixVersions.latest.doc (nixVersions.nix_2_33.doc)
  • nixVersions.latest.man (nixVersions.nix_2_33.man)
  • nixVersions.latest.tests.misc
  • nixVersions.latest.tests.nixi686
  • nixVersions.latest.tests.nixi686.dev
  • nixVersions.latest.tests.nixi686.doc
  • nixVersions.latest.tests.nixi686.man
  • nixVersions.latest.tests.nixpkgs-lib
  • nixVersions.latest.tests.pkg-config
  • nixVersions.latest.tests.simpleUefiSystemdBoot
  • nixVersions.latest.tests.srcVersion
  • nixVersions.latest.tests.upgrade

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/nix-2-33-0-released/73109/3

@xokdvium
Copy link
Contributor

We should wait for 2.33.1 to get NixOS/nix#14799. Otherwise it will get pretty messy.

@philiptaron philiptaron marked this pull request as draft December 17, 2025 04:31
@philiptaron
Copy link
Contributor Author

Drafting until then. Thanks Sergei @xokdvium

Comment on lines +206 to +218
nixComponents_2_33 = nixDependencies.callPackage ./modular/packages.nix rec {
version = "2.33.0";
inherit (self.nix_2_32.meta) maintainers teams;
otherSplices = generateSplicesForNixComponents "nixComponents_2_33";
src = fetchFromGitHub {
owner = "NixOS";
repo = "nix";
tag = version;
hash = "sha256-aVwmNDnTOYZZQbTy++rYS0NOGEu9Zwljg3+TXJmw4TE=";
};
};

nix_2_33 = addTests "nix_2_33" self.nixComponents_2_33.nix-everything;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think something else should've changed because we no longer depend on the aws-sdk.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did that a while back with a .git version bump.

@philiptaron
Copy link
Contributor Author

Any progress on a 2.33.1 @xokdvium and team? I see 2.31.3 was released a little bit ago; I'll package that up.

@xokdvium
Copy link
Contributor

xokdvium commented Jan 7, 2026

Yeah, the hydra jobsets just need a fresh rebuild before we can tag + upload.

@xokdvium
Copy link
Contributor

@philiptaron, 2.33.1 is tagged and uploaded.

@philiptaron
Copy link
Contributor Author

Thank you!

@philiptaron philiptaron marked this pull request as ready for review January 15, 2026 21:33
@Ericson2314
Copy link
Member

Ericson2314 commented Jan 15, 2026

A bunch of CI failure I can't even read, so I assume they are spurious ones stemming from the issues that appear to be going on with GitHub right now.

@philiptaron
Copy link
Contributor Author

I have the same suspicion. Thanks GitHub.

@xokdvium
Copy link
Contributor

Eh, but the eval failure seems legitimate?

@xokdvium
Copy link
Contributor

       error: function 'anonymous lambda' called with unexpected argument 'maintainers'
       at /nix/store/g4rffx2f8n9lgz7m2xwhfq8mj0w79879-source/pkgs/tools/package-management/nix/modular/packages.nix:1:1:
            1| {
             | ^
            2|   lib,

@philiptaron
Copy link
Contributor Author

I didn't rebase on master 🤦🏻

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jan 15, 2026
@Ericson2314 Ericson2314 added this pull request to the merge queue Jan 16, 2026
@Ericson2314 Ericson2314 added the backport release-25.11 Backport PR automatically label Jan 16, 2026
Merged via the queue into NixOS:master with commit 249e5cb Jan 16, 2026
48 of 52 checks passed
@Ericson2314 Ericson2314 deleted the nixVersions.nix_2_33 branch January 16, 2026 00:54
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Jan 16, 2026

Successfully created backport PR for release-25.11:

@github-actions github-actions bot added the 8.has: port to stable This PR already has a backport to the stable release. label Jan 16, 2026
@trofi
Copy link
Contributor

trofi commented Jan 16, 2026

Looks like this update broke contentAddressedByDefault = true; systems again. Simple reproducer against nixosTests:

2.33.1 fails as:

$ nix build --no-link -f. nixosTests.boot.uefiUsb --arg config '{ contentAddressedByDefault = true; }'
evaluation warning: The user 'root' has multiple of the options
                    `initialHashedPassword`, `hashedPassword`, `initialPassword`, `password`
                    & `hashedPasswordFile` set to a non-null value.

                    If multiple of these password options are set at the same time then a
                    specific order of precedence is followed, which can lead to surprising
                    results. The order of precedence differs depending on whether the
                    {option}`users.mutableUsers` option is set.

                    If the option {option}`users.mutableUsers` is
                    `true`, then the order of precedence is as shown
                    below, where values on the left are overridden by values on the right:
                    {option}`initialHashedPassword` -> {option}`initialPassword` -> {option}`hashedPassword` -> {option}`password` -> {option}`hashedPasswordFile`

                    The values of these options are:
                    * users.users."root".hashedPassword: null
                    * users.users."root".hashedPasswordFile: "/09x4c5d9qvjikiv14lni976nbgm9p8briibh7xf7pj48n7721485"
                    * users.users."root".password: null
                    * users.users."root".initialHashedPassword: ""
                    * users.users."root".initialPassword: null
error:
       … while parsing derivation '/nix/store/c363hbhjag67d1a9qxc4jdqxpiynijpq-closure-info.drv'

       error: path '/1brwbcr7s8g31j7avczw6l8rk2imx9wybac58sz6vxxgjavw6dky/modprobe.conf' is not in the Nix store

2.32.5:

nix build --no-link -f. nixosTests.boot.uefiUsb --arg config '{ contentAddressedByDefault = true; }'
evaluation warning: The user 'root' has multiple of the options
                    `initialHashedPassword`, `hashedPassword`, `initialPassword`, `password`
                    & `hashedPasswordFile` set to a non-null value.

                    If multiple of these password options are set at the same time then a
                    specific order of precedence is followed, which can lead to surprising
                    results. The order of precedence differs depending on whether the
                    {option}`users.mutableUsers` option is set.

                    If the option {option}`users.mutableUsers` is
                    `true`, then the order of precedence is as shown
                    below, where values on the left are overridden by values on the right:
                    {option}`initialHashedPassword` -> {option}`initialPassword` -> {option}`hashedPassword` -> {option}`password` -> {option}`hashedPasswordFile`

                    The values of these options are:
                    * users.users."root".hashedPassword: null
                    * users.users."root".hashedPasswordFile: "/09x4c5d9qvjikiv14lni976nbgm9p8briibh7xf7pj48n7721485"
                    * users.users."root".password: null
                    * users.users."root".initialHashedPassword: ""
                    * users.users."root".initialPassword: null
[0/3168 built, 16/358/1480 copied (134.3/14819.9 MiB), 124.7/5988.7 MiB DL] fetching comonad-5.0.9-r1.cabal from https://cache.nixos.org

builds things as expected.

@trofi
Copy link
Contributor

trofi commented Jan 16, 2026

Filed upstream nix report as NixOS/nix#15003

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

Labels

8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. backport release-25.11 Backport PR automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants