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

tools: Update iOS toolchain #2184

Merged
merged 2 commits into from
Dec 17, 2023
Merged

tools: Update iOS toolchain #2184

merged 2 commits into from
Dec 17, 2023

Conversation

julianoes
Copy link
Collaborator

@julianoes julianoes commented Nov 18, 2023

This updates the iOS toolchain from:
https://github.com/leetal/ios-cmake/blob/master/ios.toolchain.cmake as of d1aacfec (2023-11-19).

Alternative to #2176.

@julianoes
Copy link
Collaborator Author

@ThomasDebrunner could you test or debug this? I don't have a Mac to look into this.

@julianoes
Copy link
Collaborator Author

julianoes commented Nov 25, 2023

Trying to add this bugfix back in: leetal/ios-cmake@6e9021a
leetal/ios-cmake#88

Edit: Nevermind, didn't help.

This updates the iOS toolchain from:
https://github.com/leetal/ios-cmake/blob/master/ios.toolchain.cmake
as of d1aacfec (2023-11-19).

Signed-off-by: Julian Oes <[email protected]>
@julianoes julianoes force-pushed the pr-update-ios-toolchain branch from cf81f4c to d336d11 Compare December 17, 2023 04:44
@julianoes julianoes force-pushed the pr-update-ios-toolchain branch 4 times, most recently from 9388bfe to 64af7ed Compare December 17, 2023 19:26
@julianoes julianoes force-pushed the pr-update-ios-toolchain branch from b64337e to d1a9f13 Compare December 17, 2023 19:53
@julianoes
Copy link
Collaborator Author

The issue was leetal/ios-cmake#146. Now that I removed the optimization as @JonasVautherin suggested, it seems to build again, phew.

@julianoes julianoes merged commit 7839d62 into main Dec 17, 2023
27 checks passed
@julianoes julianoes deleted the pr-update-ios-toolchain branch December 17, 2023 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant