-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[release/9.0] Upgrade vendored Brotli dependency to v1.1.0 #107102
[release/9.0] Upgrade vendored Brotli dependency to v1.1.0 #107102
Conversation
/backport to release/8.0-staging |
Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/10602606754 |
Tagging subscribers to this area: @dotnet/area-system-io-compression |
@jkoritzinsky there was a patch in the main PR in a separate commit: https://github.com/dotnet/runtime/pull/106994/commits @jkotas do we care to have the commits split in this PR like you requested in my zlib-ng update? |
@carlossanlop I included those changes into the initial commit in this PR. |
This does not have any custom patches that have not been upstreamed, and the changes outside the vendored Brotli are very minimal. I do not have a problem with it all being in one commit. |
/ba-g Re-ran failed CI legs and they passed. Only one known/pre-existing failure showed up. |
Backport of #106994 to 9.0.
We want to be on the most recent release of Brotli for .NET 9.
See #106887
@carlossanlop can you get M2 approval from your side?