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
azcopy: 10.0.1 -> 10.3.2 #73613
azcopy: 10.0.1 -> 10.3.2 #73613
Conversation
Note, at one point in 's history, this built with 10.3.1. I tested this PR with 10.3.1 as well, it fails in the same way. So, it seems like something has changed in nixpkgs such that this is occuring. |
77b06cc
to
44afd4a
Compare
It appears there's precedent with another package for using the 1.12 builder. I don't really understand it, but it unblocks upgrading the build. Should be good to go. nixpkgs/pkgs/top-level/all-packages.nix Line 2617 in 78cb116
|
44afd4a
to
cff97b9
Compare
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.
Sorry for the delay. I've added a few suggestions, besides that LGTM
cff97b9
to
cb96302
Compare
Thanks @marsam for the great review! I've made those changes and have updated this branch/PR. |
cb96302
to
ab5fa1c
Compare
Thanks, hopefully this looks a bit better. |
ab5fa1c
to
0709d4e
Compare
Thanks! I've made minor changes, I hope you don't mind @GrahamcOfBorg build azure-storage-azcopy |
Upgrades
azcopy
to latest, usesbuildGo112Module
to work around issue.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @