Skip to content

Port infra branch features - #587

Open
rhendric wants to merge 3 commits into
NixOS:mainfrom
rhendric:rhendric/infra-merge
Open

rhendric wants to merge 3 commits into
NixOS:mainfrom
rhendric:rhendric/infra-merge

Conversation

@rhendric

Copy link
Copy Markdown
Member

The infra project currently references the infra branch, which holds commits that hard-code two behaviors: dropping store references to executable tools, and adding extra options to nixpkgs-review. This PR makes it possible for infra to reference the main branch instead, provided that the infra project does the following:

  • Overrides the nixpkgs-update package with withStaticToolPaths = false;
  • Adds
     --nixpkgs-review-opt 'extra-nixpkgs-config={ allowBroken = false; }' \
     --nixpkgs-review-opt num-eval-workers=3 \
     --nixpkgs-review-opt max-memory-size=4096
    
    to invocations of nixpkgs-update

When this merges, I'll provide the PR to infra to switch the reference and add these changes.

Closes #461.

Infra wants to set mmdoc to nix-systems/empty.
@rhendric
rhendric marked this pull request as ready for review October 26, 2026 01:08
In infra, we want to use:

    --nixpkgs-review-opt 'extra-nixpkgs-config={ allowBroken = false; }'
    --nixpkgs-review-opt num-eval-workers=3
    --nixpkgs-review-opt max-memory-size=4096
@rhendric
rhendric force-pushed the rhendric/infra-merge branch from 5bacfac to f65bd3f Compare October 26, 2026 12:15

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant