Skip to content

cmake: enable pthreads for BoringSSL/AWS-LC#21168

Closed
vszakats wants to merge 7 commits intocurl:masterfrom
vszakats:bssl-thread
Closed

cmake: enable pthreads for BoringSSL/AWS-LC#21168
vszakats wants to merge 7 commits intocurl:masterfrom
vszakats:bssl-thread

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Mar 31, 2026

Depends on 6828df7 #21176


  • maybe also for non-static builds, to always propagate to curl-config.cmake.
  • fix try_compile for Threads::Threads

@vszakats vszakats marked this pull request as ready for review March 31, 2026 12:06
@vszakats vszakats changed the title [WIP] cmake: enable pthreads for BoringSSL/AWS-LC static builds cmake: enable pthreads for BoringSSL/AWS-LC Mar 31, 2026
@vszakats
Copy link
Copy Markdown
Member Author

vszakats commented Mar 31, 2026

These two backends remain a lost cause or at best a dice roll for
curl-for-win.

It's toolchain bug surf. They seem to be not tested with mingw-w64 and
open source toolchains on Windows. Now the crash on startup disappeared
after a couple of years, also the pthreads _setjmp heisenbug seems to
be gone, replaced by BoringSSL seeming incompatibility with cfguard,
which happened earlier with AWS-LC, but no longer, now throwing weird
linker warnings instead, and linking/running seemingly fine. With
llvm-mingw; the other toolchains fail with various other linker errors.
(this is just Windows Intel.)

(sadly no actual commit or trace, or upstream record of these issues,
making things mushy and uncertain.)

Either way this PR is a tiny step to make them build better by default.

@vszakats vszakats closed this in f14d9e6 Mar 31, 2026
@vszakats vszakats deleted the bssl-thread branch March 31, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant