Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

[ios, macos] Update minimum deployment versions to iOS 9, macOS 10.11 #11776

Merged
merged 9 commits into from
Apr 27, 2018

Conversation

friedbunny
Copy link
Contributor

@friedbunny friedbunny commented Apr 25, 2018

As promised in #11663, this bumps the minimum deployment version of our SDKs to iOS 9 and macOS 10.11. These changes will take effect in ios-v4.1.0 and macos-v0.8.0.

To-do

Tail work

/cc @mapbox/maps-ios

@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS refactor macOS Mapbox Maps SDK for macOS labels Apr 25, 2018
@friedbunny friedbunny added this to the ios-v4.1.0 milestone Apr 25, 2018
@friedbunny friedbunny self-assigned this Apr 25, 2018
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A long time coming!

Lightweight generics are available everywhere with the bump to iOS 9 and macOS 10.11.
@friedbunny
Copy link
Contributor Author

friedbunny commented Apr 26, 2018

Ticket out switching to sqlite3_bind_blob64/ sqlite3_bind_text64, now that our SQLite version supports them.

As long as we have a single database implementation that’s platform agnostic, I don’t think we’ll ever be able to switch to these methods. Though iOS will now support them, Android (per docs) didn’t get a suitable SQLite version until maybe API 21 (and even then manufacturers could supply different versions).

Is this worth ticketing out?

/cc @jfirebaugh @kkaefer

@1ec5
Copy link
Contributor

1ec5 commented Apr 27, 2018

Note that, from the time we merge this PR, any usage of lightweight generics with Foundation collections needs to be retrofitted with compatibility macros like NS_ARRAY_OF() when backporting to the release-boba branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants