Skip to content

Revert "Put brotli on the FetchContent plan" #109108

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

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

steveisok
Copy link
Member

I think this might have broken the nativeaot legs on the official build. All nativeaot legs error out with an arch variant of:

bin/native/net10.0-linux-Release-arm/libbrotlicommon.a' is not added because the package already contains file 'runtimes/linux-arm/native/libbrotlicommon.a

Reverts #107166

Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

@steveisok
Copy link
Member Author

Kicked off an official build. If it fails the same, will close this PR.

@steveisok
Copy link
Member Author

The nativeaot legs pass on the official build, so I'm going to merge this to unblock.

@steveisok
Copy link
Member Author

/ba-g Unblocks the official build.

@steveisok steveisok merged commit b780069 into main Oct 22, 2024
86 of 142 checks passed
@akoeplinger akoeplinger deleted the revert-107166-brotli-upgrade branch October 22, 2024 12:51
jkoritzinsky added a commit to jkoritzinsky/runtime that referenced this pull request Nov 11, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants