Skip to content

Add win32_compat dependency to libraries using pthread.h - #1520

Merged
val-ms merged 1 commit into
Cisco-Talos:mainfrom
zaddach:add_win32_compat_dependency
Dec 16, 2025
Merged

Add win32_compat dependency to libraries using pthread.h#1520
val-ms merged 1 commit into
Cisco-Talos:mainfrom
zaddach:add_win32_compat_dependency

Conversation

@zaddach

@zaddach zaddach commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

A couple of intermediate libraries (such as regex, bytecode_runtime, ...) use the pthread.h header. On Windows, it is made available through the PThreadW32 library, and those libraries need to "link" against the win32_compat library to be able to see the header.

@zaddach
zaddach force-pushed the add_win32_compat_dependency branch from 0da9a36 to 5aa8d0d Compare June 19, 2025 13:32
@zaddach
zaddach force-pushed the add_win32_compat_dependency branch 4 times, most recently from 2a25035 to e5870ae Compare August 6, 2025 16:21
@val-ms
val-ms force-pushed the add_win32_compat_dependency branch from c7eff82 to 89a5860 Compare December 16, 2025 15:22
@val-ms

val-ms commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Just rebased and force-pushed to see this run through CI/CD.

@val-ms val-ms left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CI/CD (github and jenkins) look good.

@val-ms
val-ms merged commit cb844d3 into Cisco-Talos:main Dec 16, 2025
22 of 24 checks passed
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