Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Change prefix for musl targets to not use wrapper #654

Closed
wants to merge 1 commit into from

Conversation

Timmmm
Copy link

@Timmmm Timmmm commented Feb 7, 2022

This should fix cross-compilation to musl when you haven't manually set the compiler name. It's also more consistent with the existing musl targets (e.g. mips).

See #105 and https://github.com/FiloSottile/homebrew-musl-cross#:~:text=If%20you%20encounter%20issues

Tested with brew install FiloSottile/musl-cross/musl-cross on Mac.

This should fix cross-compilation to musl when you haven't manually set the compiler name. It's also more consistent with the existing musl targets (e.g. mips).

See rust-lang#105 and https://github.com/FiloSottile/homebrew-musl-cross#:~:text=If%20you%20encounter%20issues
@Timmmm
Copy link
Author

Timmmm commented Feb 7, 2022

Uhm not sure why the tests are failing and there's no error messages :-/

@m-ou-se
Copy link
Member

m-ou-se commented May 11, 2022

At least on Arch Linux, the musl package for x86-64 contains musl-gcc, but not x86_64-linux-musl-cc.

@Timmmm
Copy link
Author

Timmmm commented May 11, 2022

Yeah I figured that out eventually. IIRC it's the same on Debian based distros too unfortunately.

I guess searching for both options might be an option, or using x86_64-linux-musl-cc on all host platforms except Linux.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants