-
Notifications
You must be signed in to change notification settings - Fork 185
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
Bump the vcpkg builtin-baseline
.
#5010
Conversation
|----------------------------|---------------------------------------------------------------------------------------------------------| | ||
| `libmagic` | Updating to the upstream port deferred due to failures. | | ||
| `openssl` | Pinning to OpenSSL 1.1 until we can move to 3.0 in January 2024. | | ||
| `pcre2` | To be removed alongside libmagic. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The custom port for PCRE2 was applying some patches to its CMake build system, to make it usable from libmagic. These patches are now upstreamed.
| `pcre2` | To be removed alongside libmagic. | | ||
| `azure-storage-common-cpp` | Patching to disable default features on libxml2 (https://github.com/Azure/azure-sdk-for-cpp/pull/5221). | | ||
| `libfaketime` | Port does not yet exist upstream | | ||
| `vcpkg-cmake-config` | Patching to fix build issues with CMake 3.29.1. (https://github.com/microsoft/vcpkg/pull/38017) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Subsequent versions of CMake 3.29.x have fixed the breaking change and this custom port is no longer needed.
4bc0adb
to
d8ae79d
Compare
This is not necessary and in fact it never had the intended effect; disabling default features had to be and was made in `azure-storage-common-cpp`.
The custom AWS SDK port is no longer needed; the patch has been upstreamed to vcpkg.
d8ae79d
to
d1ddee5
Compare
I bumped the baseline again to get a more recent AWS and Google Cloud SDK version. |
This comment was marked as resolved.
This comment was marked as resolved.
a6d9d66
to
f496913
Compare
SC-48154
This PR bumps the vcpkg
builtin-baseline
which causes all of our dependencies to be updated to the most recent versions available on vcpkg. Some of our custom ports are no longer needed and were removed as well.To view the full list of updated ports, you can clone vcpkg and run
git diff 72010900b7cee36cea77aebb97695095c9358eaf 101cc9a69a1061969caf4b73579a34873fdd60fe -- .\versions\baseline.json > baseline.diff
. This is an abridged summary of the updated ports that we depend on:The bump to the baseline might cause configure errors due to the vcpkg repository being out of date In this case, try one of the following things:
VCPKG_ROOT
environment variable, update the vcpkg repository to the latest available commit, and try configuring again.build/_deps
and configure again. If it doesn't work, try performing a clean build.Let me know if any additional testing is required.
TYPE: BUILD
DESC: Update vcpkg version baseline to microsoft/vcpkg@101cc9a.