Skip to content

feat(flake-info): fallbacks#1143

Open
Malix-Labs wants to merge 1 commit intoNixOS:mainfrom
Malix-Labs:flake-info-fallback
Open

feat(flake-info): fallbacks#1143
Malix-Labs wants to merge 1 commit intoNixOS:mainfrom
Malix-Labs:flake-info-fallback

Conversation

@Malix-Labs
Copy link
Copy Markdown
Contributor

@Malix-Labs Malix-Labs commented Feb 14, 2026

This adds fallbacks to flake-info such that it will try to build only packages / apps / options if one fails

Note that "if the other fail" means that the added flakes are defectuous (e.g. many modules read config.* at evaluation time (e.g. nix-bitcoin) which is invalid when you don’t have a full NixOS configuration ; they should instead use mkIf / mkOption or read config in places that only evaluate when options are realized)

I personally don't know if this should be merged or if we instead should rightfully error when flakes are defectuous (and we just comment out the flakes that fails until they are fixed instead), RFC

@raboof
Copy link
Copy Markdown
Member

raboof commented Feb 15, 2026

I personally don't know if this should be merged or if we instead should rightfully error when flakes are defectuous (and we just comment out the flakes that fails until they are fixed instead), RFC

The risk I see with this approach is that a flake that used to work might 'break' and switch to the fallback without us noticing. I think we would like to notice and be able to fix the issue, instead of having it disappear silently.

Still, for flakes known to have this issue I think it would be nice to be able to ingest them like this on a per-flake opt-in basis.

@Malix-Labs
Copy link
Copy Markdown
Contributor Author

The risk I see with this approach is that a flake that used to work might 'break' and switch to the fallback without us noticing. I think we would like to notice and be able to fix the issue, instead of having it disappear silently.

Valid concern, shouldn't be the case indeed.

Then maybe not erroring, but throwing a warning and creating a GitHub issue linking to it?

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