-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[abseil] Update to 20240116.1 #37461
Conversation
Why do you remove the feature? |
Reviewing our maintenance guidelines does not recommend adding the C++ standard as a non-functional feature. https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/contributing/maintainer-guide.md#do-not-use-features-to-control-alternatives-in-published-interfaces |
Yes, probably it wouldn't be added today.
I don't say it works perfectly. For precise control, it is probably necessary to customize the triplet file for the toolchain and ports of interest. |
Thanks for the update! |
Fixes #37439
Delete feature cxx17.
SHA512s are updated for each updated download.The "supports" clause reflects platforms that may be fixed by this new version.Any fixed CI baseline entries are removed from that file.Any patches that are no longer applied are deleted from the port's directory../vcpkg x-add-version --all
and committing the result.Usage test pass with following triplets: