Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

autotools, darwin: restrict -force_cpusubtype_ALL switch to powerpc only #109

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sezero
Copy link
Contributor

@sezero sezero commented Jul 30, 2024

Fixes: #107
Fixes: #108
(See also: mkxp-z/mkxp-z#200)

@Splendide-Imaginarius
Copy link

Thanks for fixing this. This GitHub repo hasn't gotten any commits in 3 years. Maybe try submitting this fix to https://gitlab.xiph.org/xiph/vorbis (last commit was a month ago)?

@sezero
Copy link
Contributor Author

sezero commented Jul 30, 2024

This GitHub repo hasn't gotten any commits in 3 years. Maybe try submitting this fix to https://gitlab.xiph.org/xiph/vorbis (last commit was a month ago)?

I don't intend to, at the moment. But if you can post a notifier at there, please do so.

@sezero
Copy link
Contributor Author

sezero commented Aug 2, 2024

Thanks for fixing this. This GitHub repo hasn't gotten any commits in 3 years. Maybe try submitting this fix to https://gitlab.xiph.org/xiph/vorbis (last commit was a month ago)?

https://gitlab.xiph.org/xiph/vorbis/-/issues/2352

@Splendide-Imaginarius
Copy link

@sezero Is it likely that SDL_mixer will pull in this fix if upstream doesn't review your PR?

@sezero
Copy link
Contributor Author

sezero commented Sep 28, 2024

@sezero Is it likely that SDL_mixer will pull in this fix if upstream doesn't review your PR?

I can apply it to SDL's vendored libvorbis, but when SDL_mixer is built in vendored mode it uses cmake and not autotools (the issue and the fix is autotools-only), therefore no point in that...

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.

Building on macos arm through rosetta terminal causes ld: unknown option
2 participants