-
Notifications
You must be signed in to change notification settings - Fork 991
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
Fix macOS <-> Linux cross cflags resolution #13230
Conversation
ping |
Hi @sfackler, thanks so much for your detailed report and for opening this PR. Indeed I believe the logic was not robust enough to account for the fact that you can target Linux from macOS. I have made a small fix to your PR - I believe the conditional should cover the case where
Note for @czoido: there's a chance this doesn't fully cover the case of targetting iOS from Linux (see https://github.com/tpoechtrager/cctools-port) - but this PR at least doesn't prevent that :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to add a test for this.
…etting non-Apple OS
Changelog: Fix: Fix flags passed by AutotoolsToolchain when cross compiling from macOS to a non-Apple OS.
Docs: Omit
Fixes #13219
develop
branch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.