Skip to content

Eval warnings#15272

Merged
xokdvium merged 2 commits intoNixOS:masterfrom
roberth:eval-warnings
Feb 25, 2026
Merged

Eval warnings#15272
xokdvium merged 2 commits intoNixOS:masterfrom
roberth:eval-warnings

Conversation

@roberth
Copy link
Member

@roberth roberth commented Feb 17, 2026

Motivation

We've been warned.

Context


Add 👍 to pull requests you find important.

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

It is only supported on that host platform.
This adds a bunch of redundant-ish wiring logic, because the alternative
is accessing `meta.platforms` generically, but that would cause these
flake output attrsets *containing* the packages to be *strict in all
packages*, which is very bad for performance and robustness.

Checked:
- Cross-compilation still works for Linux targets (arm, riscv),
  even when build platform is Darwin
- No attributes for nix-nswrapper for native Darwin
@roberth roberth requested a review from edolstra as a code owner February 17, 2026 15:47
@xokdvium xokdvium added this pull request to the merge queue Feb 25, 2026
Merged via the queue into NixOS:master with commit 76eca8f Feb 25, 2026
25 of 26 checks passed
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.

2 participants