nixops: remove (v1.x) #150059
Closed
nixops: remove (v1.x) #150059
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The NixOps 1.x branch has been unsupported for a long time, breaks very often in Nixpkgs, because it requires heavily outdated deps, and has put undue burden on the python packages maintainers. The 1.x branch is unsustainable and it is time that we acknowledge this fact. The only way forward for NixOps is the unreleased version 2.0, which needs testing and fixes before it can be released and become the replacement for the default nixops package.
ofborg
bot
added
10.rebuild-darwin: 0
Mindavi
reviewed
Dec 11, 2021
| @@ -32548,7 +32548,7 @@ with pkgs; | |||
|
|
|||
| nixStatic = pkgsStatic.nix; | |||
|
|
|||
| nixops = callPackage ../tools/package-management/nixops { }; | |||
| nixops = builtins.throw "The NixOps 1.x branch is unmaintained and unsupported. Please upgrade to nixopsUnstable and contribute to the NixOps 2 release."; | |||
There was a problem hiding this comment.
This can go in aliases, I think.
There was a problem hiding this comment.
I don't see why that would be useful. Isn't it nicer to keep the nixops attrs together? Will make the 2.0 release slightly easier.
There was a problem hiding this comment.
We use aliases to prevent all-packages.nix from being cluttered with 100's of deprecation/removal errors.
|
Any updates on this? |
|
Fwiw: I think this is sad and a testament to how badly Python package maintenance works in nixpkgs. |
adisbladis
added a commit
to adisbladis/nixpkgs
that referenced
this pull request
Jan 12, 2022
The NixOps derivation keeps breaking because of unrelated Python packaging work and is incredibly difficult to keep alive. Rather than outright removing 1.7 as suggested in NixOS#150059 this change shuold make it much easier to keep 1.7 on life support without causing any burden for those wanting to work on python3 only.
13 tasks
|
Solved by #154715 Thanks all! |
jsoo1
pushed a commit
to awakesecurity/nixpkgs
that referenced
this pull request
Oct 10, 2022
The NixOps derivation keeps breaking because of unrelated Python packaging work and is incredibly difficult to keep alive. Rather than outright removing 1.7 as suggested in NixOS#150059 this change shuold make it much easier to keep 1.7 on life support without causing any burden for those wanting to work on python3 only.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation for this change
NOTE: a preferable solution may be feasible #154715
The NixOps 1.x branch has been unsupported for a long time, breaks
very often in Nixpkgs, because it requires heavily outdated deps,
and has put undue burden on the python packages maintainers.
The 1.x branch is unsustainable and it is time that we acknowledge
this fact. The only way forward for NixOps is the unreleased
version 2.0, which needs testing and fixes before it can be
released and become the replacement for the default nixops package.
As mentioned before: #144627 (comment)
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)nixos/doc/manual/md-to-db.shto update generated release notescc @AmineChikhaoui @eelco @rob