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

[ios] Cherry-pick #13427 - Expose "local ideograph font family" in map snapshotters #13547

Merged
merged 6 commits into from
Jan 8, 2019

Conversation

jmkiley
Copy link
Contributor

@jmkiley jmkiley commented Dec 11, 2018

This PR cherry-picks #13427 into the iOS 8 branch.

cc @lloydsheng @chriswu42

@jmkiley jmkiley added the iOS Mapbox Maps SDK for iOS label Dec 11, 2018
@jmkiley jmkiley self-assigned this Dec 11, 2018
Copy link
Contributor

@ChrisLoer ChrisLoer left a comment

Choose a reason for hiding this comment

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

LGTM. Are all those linux builds just broken on our ios8 branch?

@jmkiley
Copy link
Contributor Author

jmkiley commented Dec 17, 2018

@ChrisLoer Yeah, it looks like a higher version of cmake is required than is in the branch. I'm not sure if there is something I can bring into the branch so that they can pass.

@ChrisLoer
Copy link
Contributor

@kkaefer Do you have any pointers for getting the linux builds running on the ios8 branch again? It also seems somewhat reasonable to just ignore or disable these builds, since this branch is only used for iOS (although the sanitize builds are meant to catch errors that could show up on any platform).

Copy link
Contributor

@fabian-guerra fabian-guerra left a comment

Choose a reason for hiding this comment

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

LGTM

@kkaefer
Copy link
Member

kkaefer commented Jan 2, 2019

918e571 should fix this build

@jmkiley jmkiley closed this Jan 7, 2019
@jmkiley jmkiley reopened this Jan 7, 2019
@jmkiley
Copy link
Contributor Author

jmkiley commented Jan 8, 2019

Thanks @kkaefer!

@jmkiley jmkiley merged commit e3d622e into ios8-release-gazpacho Jan 8, 2019
@jmkiley jmkiley deleted the jmkiley-cp-13427 branch January 8, 2019 00:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants