Skip to content

Conversation

@dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Sep 26, 2025

Export -pthread and -lpthread as is. Fixes #1595.
Cleanup.

@dg0yt dg0yt marked this pull request as ready for review September 26, 2025 04:53
@1480c1
Copy link

1480c1 commented Sep 26, 2025

As far as I can tell, this would not actually fix the linked issue as the flag is -lpthread in that case, not -pthread.

I've seen some other cmake projects that instead does two checks, first for if it's an absolute path, then one for if it looks like a flag (either starts with - or they check a bit more specific on -l vs -f vs no prefix). I'm not sure

@dg0yt

This comment was marked as outdated.

@dg0yt
Copy link
Contributor Author

dg0yt commented Sep 26, 2025

Wait, I see what you mean.

@dg0yt dg0yt marked this pull request as draft September 26, 2025 05:15
@dg0yt dg0yt marked this pull request as ready for review September 26, 2025 05:23
@rouault rouault added this to the 2.5.5 milestone Sep 26, 2025
@rouault rouault merged commit 1ad9bec into uclouvain:master Sep 26, 2025
12 checks passed
@dg0yt dg0yt deleted the pkgconfig branch September 26, 2025 14:12
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.

Incorrect .pc file generate for clang64 with msys2

3 participants