Skip to content

Conversation

@jjhelmus
Copy link
Contributor

@jjhelmus jjhelmus commented Dec 9, 2025

closes #829

# tcl misbehaves when performing static musl builds
# `checking whether musl-clang accepts -g...` fails with a duplicate definition error
TARGET_TRIPLE="$(echo "${TARGET_TRIPLE}" | sed -e 's/-unknown-linux-musl/-unknown-linux-gnu/g')"
# tcl will use an internal implementation of certain POSIX function when
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file is a mixture of tabs and spaces for indentation. I keep the existing mixture.

@jjhelmus
Copy link
Contributor Author

To expand a bit on this change. When compiling for musl with a static option the check for a proper strtoul implementation short circuits to unknown because of the cross compiling status (building on gnu, target is musl). This results in the substitute strtoul function being included in the build. This in turn causes a duplicate definition error because strtoul is already provided by libc.

@jjhelmus jjhelmus merged commit f322ad6 into astral-sh:main Dec 10, 2025
538 of 540 checks passed
jjhelmus added a commit to jjhelmus/python-build-standalone that referenced this pull request Dec 10, 2025
--print-multiarch reporting an incorrect triple was a bug in LLVM that
was fixed. With the project now using LLVM 21+ (astral-sh#916) the patches to
remove this check during configuration are no necessary.

closes astral-sh#865
@jjhelmus jjhelmus mentioned this pull request Dec 10, 2025
jjhelmus added a commit that referenced this pull request Dec 10, 2025
--print-multiarch reporting an incorrect triple was a bug in LLVM that
was fixed. With the project now using LLVM 21+ (#916) the patches to
remove this check during configuration are no necessary.

closes #865
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.

Upgrade musl builds to latest LLVM

2 participants