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

Update to Xcode 15 #2107

Merged
merged 4 commits into from
May 21, 2024
Merged

Conversation

andyfinnell
Copy link
Collaborator

Summary

We want to make Texture build with recent tools like Xcode 15. Part of this is bumping the minimum supported OS to iOS 14.

I only removed the dead code (i.e. code gated on versions older than iOS 14). There are still some warnings about outdated APIs in use, but they are non-trivial changes to fix. Also, several of the examples rely on remote media (images or videos) that no longer exist. If they were trivially fixable, I fixed them. If not, I left them.

Test plan

Run all the examples and verify it works as I expect. Run all the unit tests and make them pass.

## Summary

We want to make Texture build with recent tools like Xcode 15. Part of this is bumping the minimum supported OS to iOS 14.

I only removed the dead code (i.e. code gated on versions older than iOS 14). There are still some warnings about outdated APIs in use, but they are non-trivial changes to fix.

## Test plan

Run all the examples. Run all the unit tests and make them pass.
@CLAassistant
Copy link

CLAassistant commented May 17, 2024

CLA assistant check
All committers have signed the CLA.

@andyfinnell andyfinnell merged commit e9bd484 into TextureGroup:master May 21, 2024
11 checks passed
@andyfinnell andyfinnell deleted the update-to-xcode-15 branch May 21, 2024 17:04
@mattxw
Copy link

mattxw commented May 24, 2024

Out of curiosity, why do Texture need to bump the minimum iOS version to 14?

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.

None yet

4 participants