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

Workaround for wrong SDK in build process for Xcode15 #13

Merged
merged 1 commit into from
Oct 8, 2023

Conversation

davidtaylor-juul
Copy link
Contributor

After upgrading to Xcode15 we started seeing this warning: using sysroot for 'iPhoneOS' but targeting 'MacOSX' followed by a bunch of C compilation errors and the build would fail.

Added a workaround that injects the sysroot to the underlying C compiler which feels like it is a bug in swift build as one would expect the -sdk parameter to propagate down the toolchain, but it appears it does not.

Fixes #11

@twyatt twyatt mentioned this pull request Oct 6, 2023
@twyatt
Copy link
Contributor

twyatt commented Oct 6, 2023

We also tested this change against Xcode 14.3.1 (and it still works on the older version).

@ttypic
Copy link
Owner

ttypic commented Oct 8, 2023

Great! Thank you very much!

@ttypic ttypic merged commit eeaa886 into ttypic:main Oct 8, 2023
@twyatt
Copy link
Contributor

twyatt commented Oct 9, 2023

Thanks @ttypic for the quick turnaround/release. ❤️

@dusiema
Copy link

dusiema commented Oct 9, 2023

Cool! Thank you! @twyatt @ttypic

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.

Invalid Manifest (xCode 15)
4 participants