Skip to content

Conversation

staticlibs
Copy link
Collaborator

This change adds _DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR flag for Windows builds that prevents crashes when the host process, into which the ODBC driver is loaded, is run using older versions of C++ stdlib.

Details: duckdb/duckdb#17991

Testing: additional test run added that uses msvcp140.dll from VS2019 instead of the system-provided one.

@staticlibs staticlibs force-pushed the windows_mutex_fix branch 6 times, most recently from 4cddae0 to 843d735 Compare June 20, 2025 22:19
This change adds `_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR` flag for Windows
builds that prevents crashes when the host process, into which the
ODBC driver is loaded, is run using older versions of C++ stdlib.

Details: duckdb/duckdb#17991

Testing: additional test run added that uses `msvcp140.dll` from VS2019
instead of the system-provided one.
@maiadegraaf
Copy link
Collaborator

Useful change! Thanks :)

@maiadegraaf maiadegraaf merged commit 9652317 into duckdb:main Jun 25, 2025
7 checks passed
@staticlibs staticlibs deleted the windows_mutex_fix branch June 25, 2025 09:24
staticlibs added a commit to staticlibs/duckdb-odbc that referenced this pull request Jul 3, 2025
This is a backport of the PR duckdb#169 to `v1.3-ossivalis` stable branch.

This change adds `_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR` flag for Windows
builds that prevents crashes when the host process, into which the
ODBC driver is loaded, is run using older versions of C++ stdlib.

Details: duckdb/duckdb#17991

Testing: additional test run added that uses `msvcp140.dll` from VS2019
instead of the system-provided one.
staticlibs added a commit that referenced this pull request Jul 3, 2025
This is a backport of the PR #169 to `v1.3-ossivalis` stable branch.

This change adds `_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR` flag for Windows
builds that prevents crashes when the host process, into which the
ODBC driver is loaded, is run using older versions of C++ stdlib.

Details: duckdb/duckdb#17991

Testing: additional test run added that uses `msvcp140.dll` from VS2019
instead of the system-provided one.
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.

2 participants