Skip to content

Musl build 04 conditionals - #4673

Open
peppergrayxyz wants to merge 20 commits into
ptitSeb:mainfrom
peppergrayxyz:musl_build_04_conditionals
Open

peppergrayxyz wants to merge 20 commits into
ptitSeb:mainfrom
peppergrayxyz:musl_build_04_conditionals

Conversation

@peppergrayxyz

Copy link
Copy Markdown
Contributor

Part of #4580

@peppergrayxyz
peppergrayxyz force-pushed the musl_build_04_conditionals branch 2 times, most recently from 8a0d530 to 60f60e3 Compare November 10, 2026 10:38
@ptitSeb

ptitSeb commented Nov 12, 2026

Copy link
Copy Markdown
Owner

And you need to rebase now

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
my32_pthread_mutex_init: fake->i386__kind:
incomplete definition of type 'struct __pthread_mutex_s'

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Signed-off-by: Pepper Gray <hello@peppergray.xyz>
@peppergrayxyz
peppergrayxyz force-pushed the musl_build_04_conditionals branch from 60f60e3 to e9d0d5c Compare November 12, 2026 10:41
Comment thread CMakeLists.txt

include(CheckIncludeFile)

set(PROJECT_HEADERS

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Maybe all those checks should go in another file and just include it here?

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.

second this. also is it necessary to enable these checks for non-musl targets?

@ptitSeb
ptitSeb requested a review from ksco November 12, 2026 10:52
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.

3 participants