-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
nixos-option needs rewriting #293543
Copy link
Copy link
Closed
Labels
1.severity: significantNovel ideas, large API changes, notable refactorings, issues with RFC potential, etc.Novel ideas, large API changes, notable refactorings, issues with RFC potential, etc.6.topic: nixosIssues or PRs affecting NixOS modules, or package usability issues specific to NixOSIssues or PRs affecting NixOS modules, or package usability issues specific to NixOS
Metadata
Metadata
Assignees
Labels
1.severity: significantNovel ideas, large API changes, notable refactorings, issues with RFC potential, etc.Novel ideas, large API changes, notable refactorings, issues with RFC potential, etc.6.topic: nixosIssues or PRs affecting NixOS modules, or package usability issues specific to NixOSIssues or PRs affecting NixOS modules, or package usability issues specific to NixOS
Fields
Give feedbackNo fields configured for issues without a type.
Issue description
I had the displeasure of touching this code in #293542, and it is obvious that this tool has utterly bitrotted. I am not convinced it should be using the C++ API to begin with.
It really should be rewritten to use the CLI or C API. All the info is there and can absolutely be acquired via the CLI with a clever bit of Nix scripting.
Technical details
Please run
nix-shell -p nix-info --run "nix-info -m"and paste the result.