-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename nixos-rebuild
to nixos
#122715
Comments
nixos-rebuild
to nixos
If so, then i'm against this, i would rather see a completely new tool, like with the
|
It opens up the possibility to bring in additional subcommands in the future with the minimal amount of work needed to get there.
I agree, but how does renaming prevent that? |
Because it is much easier to push breaking changes in the new WIP tool (like changing command names, structure, behaviour, etc.), while still keeping the old one - it would be a missed opportunity to not take advantage of this, nothing more. |
I agree that, in the long term, we should have a OTOH, we don't want perfect be the enemy of good, and considering #54188 is a few years old and hasn't seen progress, I wonder if we should just do what we can with the resources we have now (sounds a bit pessimistic maybe). @domenkozar how do you see a migration from |
Looking at #54188, I see that we could do 90% of what's listed there without breaking backward compatibility. So if we can give up that 10% and do this incrementally it has a higher chance of happening. |
@domenkozar what's the 10% we'd have to drop? I'd like to point out that the current bash script is pretty spaghetti-like as it is, with the handling of flake and legacy modes, so I don't know how much more complexity it can take, before a refactor. |
I marked this as stale due to inactivity. → More info |
This issue has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nixpkgs-cli-working-group/30517/2 |
Given that Nix commands are now all subcommands of
nix
CLI, it feels appropriate (with respect to user expectations) to do the same withnixos
command.Another reason is that some commands have little to do with rebuilding and there's no inherent reason they shouldn't be part of the toolkit. A good existing example is
nixos-rebuild edit
.(keep around nixos-rebuild for backwards compatibility)
The text was updated successfully, but these errors were encountered: