Skip to content

tools: include types.h to build probes tool on musl - #11074

Open
eholzbach wants to merge 1 commit into
thesofproject:mainfrom
eholzbach:musl
Open

tools: include types.h to build probes tool on musl#11074
eholzbach wants to merge 1 commit into
thesofproject:mainfrom
eholzbach:musl

Conversation

@eholzbach

@eholzbacheholzbach commented Aug 8, 2026

Copy link
Copy Markdown
  • tools: include sys/types.h explicitly to build on musl

@sofci

sofci commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

reply test this please to run this test once

@kv2019ikv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you @eholzbach for the contribution. Can you update the git commit description to match project style and add a proper sign-off (see https://thesofproject.github.io/latest/contribute/contribute_guidelines.html ).

sys/types.h is automatically included with glibc, not musl
Signed-off-by: eholzbach <ericholzbach@pm.me>
@eholzbacheholzbach changed the title include types.h to build probes tool on musltools: include types.h to build probes tool on muslAug 10, 2026
@eholzbach
eholzbach requested a review from kv2019iAugust 10, 2026 13:19
@kv2019i

Copy link
Copy Markdown
Collaborator

test this please

@kv2019ikv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks, let me trigger the CI flows

Sign up for freeto 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.

3 participants

@eholzbach@sofci@kv2019i