Skip to content

nix/develop: Strip outputChecks when structuredAttrs is enabled#14306

Merged
xokdvium merged 1 commit intoNixOS:masterfrom
corngood:develop-structuredAttrs-fix
Oct 20, 2025
Merged

nix/develop: Strip outputChecks when structuredAttrs is enabled#14306
xokdvium merged 1 commit intoNixOS:masterfrom
corngood:develop-structuredAttrs-fix

Conversation

@corngood
Copy link
Contributor

Motivation

nix develop doesn't remove ouputChecks from the environment derivation when structured attributes are used, which causes the environment build to fail

Context

Fixes: #12917


Add 👍 to pull requests you find important.

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

@corngood corngood requested a review from edolstra as a code owner October 20, 2025 18:42
@github-actions github-actions bot added the new-cli Relating to the "nix" command label Oct 20, 2025
@Ericson2314
Copy link
Member

@corngood If you could also write a regression test (new PR is fine) that would be wonderful!

(Maybe Lix has a test for this? @xokdvium says they also fixed the bug recently.)

Merged via the queue into NixOS:master with commit 67f5cb9 Oct 20, 2025
16 checks passed
@corngood corngood deleted the develop-structuredAttrs-fix branch October 20, 2025 21:35
@corngood
Copy link
Contributor Author

@corngood If you could also write a regression test (new PR is fine) that would be wonderful!

#14312

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

Labels

new-cli Relating to the "nix" command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nix develop nixpkgs#postgresql fails due to outputChecks

3 participants