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
curl: 7.70.0 -> 7.71.0 #91399
curl: 7.70.0 -> 7.71.0 #91399
Conversation
Heh, I just realized this is going to generate a mass rebuild ><. Rebasing on staging in a minute. |
0b2e5f5
to
4e8fc3c
Compare
Patching sounds more suitable for 20.03. Why not just update to 7.71.0 here? (released today) |
Except then we need to decide whether we want to pursue #86999, which is waiting for a curl bump to 20.03. |
Oh, OK; good to know. Anyway, I meant mainly the other part of implication – that I can't see why patch in master/staging. |
I mostly did this out of shyness. I'm totally unfamiliar with the curl
package. I know it's tightly coupled with Nix but I never dive into that
part so far., I was afraid of doing something stupid.
Let's bump it.
|
I believe we update it quite routinely. |
4e8fc3c
to
36f61e7
Compare
I tried to apply the patches on top of The patches cannot be cleanly applied. Meaning we either want to bump that to I don't really have time to dig into that. Could somebody else have a look? |
I will give it a spin. |
I just merged a slightly different commit hash than any I see here: 277dd15. I'm not sure why exactly it happened, but the resulting tree is the same... |
Motivation for this change
Changelog:
https://daniel.haxx.se/blog/2020/06/24/curl-7-71-0-blobs-and-retries/
Fixes:
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)