Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@c-herrewijn
Copy link
Member

@c-herrewijn c-herrewijn commented Jul 14, 2025

Manage overlay ports via central registry: https://github.com/duckdb/vcpkg-duckdb-ports
This is part of a process to homogenize extension CI
https://github.com/duckdblabs/duckdb-internal/issues/5307

@carlopi
Copy link
Contributor

carlopi commented Jul 14, 2025

I think this makes a lot of sense, thanks.

Could you maybe edit PR message with some context? Then it's ready

@carlopi carlopi requested a review from Maxxen July 14, 2025 13:47
carlopi added a commit to carlopi/duckdb that referenced this pull request Jul 14, 2025
@Mytherin Mytherin merged commit bfb2cba into duckdb:main Jul 15, 2025
22 checks passed
carlopi added a commit to carlopi/duckdb that referenced this pull request Jul 15, 2025
Mytherin added a commit to duckdb/duckdb that referenced this pull request Jul 15, 2025
…lib (#18205)

Two not so independent changes:
* bump `vcpkg-duckdb-ports` commit to include
duckdb/vcpkg-duckdb-ports@fcdaf72,
that adds:
```cmake
set(CMAKE_CXX_WINDOWS_FLAGS
      "${CMAKE_CXX_WINDOWS_FLAGS} /D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR")
```
to `vcpkg-cmake` port.
* cherry-picks a tests by @staticlibs test to check whether correct
flags are passed down to VCPKG (part of
#18149)
* remove `vss` and `azure` (PR that adapt them are ready, needs to be
merged and then bumped, see duckdb/duckdb-vss#64
and duckdb/duckdb-azure#111)

If Windows CI for this PR passes, then good to be merged.
@c-herrewijn c-herrewijn deleted the remove-overlay-ports branch July 16, 2025 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants