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: Add brotli support #32165
Curl: Add brotli support #32165
Conversation
88636b3
to
af1a1ed
Compare
Both dependencies are in staging now. |
Already rebased on top of staging. Thanks! |
af1a1ed
to
8d479c0
Compare
Defaults: is there some estimate what (deployed) servers support brotli ATM? |
Have you really tested this? For me the log looks OK, but it doesn't retail any reference to brotli... |
@vcunat Yes. Tested and it works fine. https://gist.github.com/adisbladis/ccc6cca83fa9ae7835ba6ef36d842d36 https://gist.github.com/adisbladis/ccc6cca83fa9ae7835ba6ef36d842d36#file-curl-brotli-test-L54 and https://gist.github.com/adisbladis/ccc6cca83fa9ae7835ba6ef36d842d36#file-curl-brotli-test-L69 indicates brotli is in use. |
|
I was looking wrong. It does get into closure. I haven't yet fully got comfortable with nix-1.12 UI, apparently. |
Adds
brotli
compression support to curlDepends on these two PRs:
#32161(merged!)#32123(merged!)When these are merged and the stars are aligned I will rebase this one.(done!)While I would like to enable
brotli
by default it's a bit tricky because of circular dependencies.Motivation for this change
Adds brotli support to curl
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)