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
awscli: 1.18.150 -> 1.18.185 #104816
awscli: 1.18.150 -> 1.18.185 #104816
Conversation
I'm not sure if I've done the different version of the |
Is it possible to overwrite the version and hash of the original coloroma package or use an newer version without causing issues? |
It seems that a quite a few other packages depend on it, and I'm moving backwards by a patch number, so I thought it was probably wise to do it only for this specific package. Using a newer version won't fit the requirements for the |
That did not go as planned. Sorry about the pings. |
I'd be tempted to patch the |
Thanks, @lukegb, I'll look into that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 commit per expressions/package modified
pkgs/top-level/python-packages.nix
Outdated
@@ -1273,6 +1273,8 @@ in { | |||
|
|||
colorama = callPackage ../development/python-modules/colorama { }; | |||
|
|||
colorama043 = callPackage ../development/python-modules/colorama/0.4.3.nix { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please call this directly from awscli
pkgs/tools/admin/awscli/default.nix
Outdated
@@ -38,7 +38,7 @@ in with py.pkgs; buildPythonApplication rec { | |||
bcdoc | |||
s3transfer | |||
six | |||
colorama | |||
colorama043 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find it highly unlikely that it is incompatible with a different patch release version. Please test patching out the pinning first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, will do, just waiting to have the time.
@lovek323 Please fix the merge conflict. |
also please address @FRidh comments |
Looks like this has already been fixed. Thanks for all your help, but I'm closing for now. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)