autotools, darwin: restrict -force_cpusubtype_ALL switch to powerpc only

Open sezero opened this issue 1 year ago • 5 comments

Fixes: https://github.com/xiph/vorbis/issues/107 Fixes: https://github.com/xiph/vorbis/pull/108 (See also: https://github.com/mkxp-z/mkxp-z/pull/200)

Jul 30 '24 14:07 sezero

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)?

Jul 30 '24 23:07 Splendide-Imaginarius

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.

Jul 30 '24 23:07 sezero

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

Aug 02 '24 14:08 sezero

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

Sep 28 '24 23:09 Splendide-Imaginarius

@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...

Sep 28 '24 23:09 sezero