Skip to content

Reapply "Use the hash modulo in the derivation outputs"#15296

Merged
Mic92 merged 1 commit intoNixOS:masterfrom
Mic92:fix-nix-build
Feb 19, 2026
Merged

Reapply "Use the hash modulo in the derivation outputs"#15296
Mic92 merged 1 commit intoNixOS:masterfrom
Mic92:fix-nix-build

Conversation

@Mic92
Copy link
Member

@Mic92 Mic92 commented Feb 19, 2026

This reverts commit 4f91e95.

This broke:

when I applied this pr, --print-out-paths wouldn't print anything:

shell-for-nix-env % nix build --print-out-paths .#legacyPackages.aarch64-darwin.homeConfigurations.macos.activationPackage

After dropping the patches from my fork, it does again.

shell-for-nix-env % nix build --print-out-paths .#legacyPackages.aarch64-darwin.homeConfigurations.macos.activationPackage
/nix/store/s8mlcalszdml0v8172w4hwqnx0m6477r-home-manager-generation

Motivation

Context


Add 👍 to pull requests you find important.

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

This reverts commit 4f91e95.

This broke:

when I applied this pr, `--print-out-paths` wouldn't print anything:

```
shell-for-nix-env % nix build --print-out-paths .#legacyPackages.aarch64-darwin.homeConfigurations.macos.activationPackage
```

After dropping the patches from my fork, it does again.
```
shell-for-nix-env % nix build --print-out-paths .#legacyPackages.aarch64-darwin.homeConfigurations.macos.activationPackage
/nix/store/s8mlcalszdml0v8172w4hwqnx0m6477r-home-manager-generation
```
@github-actions github-actions bot added documentation new-cli Relating to the "nix" command with-tests Issues related to testing. PRs with tests have some priority store Issues and pull requests concerning the Nix store labels Feb 19, 2026
@Mic92
Copy link
Member Author

Mic92 commented Feb 19, 2026

Somehow I wasn't able to reproduce the bug in #12464 (comment) after my second testing round. But now it's back.

@Mic92 Mic92 enabled auto-merge February 19, 2026 00:58
@Mic92
Copy link
Member Author

Mic92 commented Feb 19, 2026

I will try to come up with a regression test next time.

@xokdvium
Copy link
Contributor

Sure. Probably best left for the next release cycle.

@Mic92 Mic92 added this pull request to the merge queue Feb 19, 2026
Merged via the queue into NixOS:master with commit 360ff05 Feb 19, 2026
15 checks passed
@Mic92 Mic92 deleted the fix-nix-build branch February 19, 2026 02:21
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 release-blocker store Issues and pull requests concerning the Nix store with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants