-
Notifications
You must be signed in to change notification settings - Fork 28
Fix Xcode warnings #52
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
Conversation
Xcode also warns because IPHONEOS_DEPLOYMENT_TARGET is not iOS 12. Well, warn as much as you want, Xcode: this pull requests does not change the requirements of this repository.
I'm not enough aware of the projects that use swiftlyfalling/SQLiteLib, and how many would break if we would raise IPHONEOS_DEPLOYMENT_TARGET to 12.
So I will eventually decide, on my own, to raise the iOS deployment target of this project. Fighting Xcode and the Apple policy on backward compatibility is not a position that one can hold for long.
I wait for the author and users of this library to chime in and express their eventual rationale against such a bump.
Xcode insists on bumping IPHONEOS_DEPLOYMENT_TARGET, but this is not acceptable for this project which has lower minimum requirements. So there will remain warnings.
4a21b3a to
fe05b40
Compare
Uh oh!
There was an error while loading. Please reload this page.
This PR removes as many Xcode warnings as possible:
Xcode also warns because IPHONEOS_DEPLOYMENT_TARGET is not iOS 12. Well, warn as much as you want, Xcode: this pull requests does not change the requirements of this repository.