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

Import required frameworks with static framework umbrella #4029

Merged
merged 2 commits into from
Feb 19, 2016

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Feb 18, 2016

This change corrects a regression introduced in #3183, when the umbrella header used in version 3.0.1 was removed. The umbrella header inside the static framework now imports the required iOS SDK frameworks. With this change, it’s no longer necessary to manually link any frameworks other than Mapbox.framework.

/cc @friedbunny

1ec5 added 2 commits February 18, 2016 14:41
The umbrella header inside the static framework now imports the required iOS SDK frameworks so manual linking is unnecessary. This change corrects a regression introduced in #3183, when the umbrella header used in version 3.0.1 was removed.
Even these three are no longer needed, now that the umbrella header pulls them in.
@1ec5 1ec5 added iOS Mapbox Maps SDK for iOS build labels Feb 18, 2016
@1ec5 1ec5 self-assigned this Feb 18, 2016
@1ec5 1ec5 added this to the ios-v3.1.1 milestone Feb 18, 2016
@1ec5 1ec5 modified the milestones: ios-v3.1.2, ios-v3.1.1 Feb 18, 2016
@1ec5 1ec5 merged commit 9f4e8dd into release-ios-v3.1.0 Feb 19, 2016
@1ec5 1ec5 deleted the 1ec5-static-umbrella branch February 19, 2016 02:00
1ec5 added a commit that referenced this pull request Feb 22, 2016
…ions

Backed out documentation about #4029 because it doesn’t apply when using a storyboard, and the installation instructions use a storyboard.
@1ec5
Copy link
Contributor Author

1ec5 commented Feb 22, 2016

This change was partially backed out in 7bed2ce. If you start out with a storyboard and the static framework, you won’t be able to link the project until you import the module. To work around that, you need to link the various and sundry framework dependencies.

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

Successfully merging this pull request may close these issues.

2 participants