Skip to content

Conversation

staticlibs
Copy link
Contributor

This change adds _DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR preprocessor definition to Python wheel builds on Windows.

For details see #17991, where the same definition was added for CLI builds.

Fixes: #17971

This change adds `_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR` preprocessor
definition to Python wheel builds on Windows.

For details see duckdb#17991, where the same definition was added for CLI
builds.

Fixes: duckdb#17971
@duckdb-draftbot duckdb-draftbot marked this pull request as draft July 9, 2025 22:48
@staticlibs staticlibs marked this pull request as ready for review July 9, 2025 22:49
Copy link
Contributor

@carlopi carlopi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I think there is no real way to test properly in CI setup than merge, and then check manually on the first built 1.3.3-dev builds.

Thanks!

@Mytherin Mytherin merged commit 117a755 into duckdb:v1.3-ossivalis Jul 10, 2025
22 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Jul 12, 2025
Allow explicit compression for user types (duckdb/duckdb#18219)
Skip logging test for smaller block sizes (duckdb/duckdb#18201)
Add VS2019 compat flag to Python wheel build (duckdb/duckdb#18198)
Bump Julia to v1.3.2 (duckdb/duckdb#18185)
Bump spatial again: include re-linking for handling global objects in Wasm (duckdb/duckdb#18170)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Jul 12, 2025
Allow explicit compression for user types (duckdb/duckdb#18219)
Skip logging test for smaller block sizes (duckdb/duckdb#18201)
Add VS2019 compat flag to Python wheel build (duckdb/duckdb#18198)
Bump Julia to v1.3.2 (duckdb/duckdb#18185)
Bump spatial again: include re-linking for handling global objects in Wasm (duckdb/duckdb#18170)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants