Skip to content

KNOWN_BUGS: remove stale Threads::Threads entry#21734

Merged
cmeister2 merged 1 commit into
masterfrom
cm2/cleanupstale
May 24, 2026
Merged

KNOWN_BUGS: remove stale Threads::Threads entry#21734
cmeister2 merged 1 commit into
masterfrom
cm2/cleanupstale

Conversation

@cmeister2
Copy link
Copy Markdown
Contributor

@cmeister2 cmeister2 commented May 24, 2026

The old CMake bug about exporting -lpthread instead of Threads::Threads
no longer matches current master. As of
2d546d2 ("cmake: use Threads::Threads
imported target for POSIX Threads"), the build now uses Threads::Threads
and the generated CMake package config resolves the dependency
explicitly, so this KNOWN_BUGS entry is stale.

Ref: 2d546d2 #21163

@cmeister2 cmeister2 requested a review from vszakats May 24, 2026 08:04
@vszakats vszakats marked this pull request as ready for review May 24, 2026 08:26
The old CMake bug about exporting -lpthread instead of Threads::Threads
no longer matches current master. As of 2d546d2
("cmake: use Threads::Threads imported target for POSIX Threads"),
the build now uses Threads::Threads and the generated CMake package
config resolves the dependency explicitly, so this KNOWN_BUGS entry is stale.

Closes #21734
@cmeister2
Copy link
Copy Markdown
Contributor Author

added Closes to commit message.

@cmeister2 cmeister2 merged commit 307cfd0 into master May 24, 2026
91 checks passed
@cmeister2 cmeister2 deleted the cm2/cleanupstale branch May 24, 2026 09:12
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
The old CMake bug about exporting -lpthread instead of Threads::Threads
no longer matches current master. As of 2d546d2
("cmake: use Threads::Threads imported target for POSIX Threads"),
the build now uses Threads::Threads and the generated CMake package
config resolves the dependency explicitly, so this KNOWN_BUGS entry is stale.

Closes curl#21734
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.

2 participants