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

Podspec fix for people using CocoaPods for distribution and not NPM #1504

Closed
wants to merge 2 commits into from

Conversation

orta
Copy link
Contributor

@orta orta commented Feb 6, 2019

Typically during the install process npm calls node ./scripts/download-mapbox-gl-native-ios-if-on-mac.js 3.7.8 to download the MapBox SDK: here

However, when you install this library via CocoaPods (when separating your JS / native code ) then this script doesn't run. This allows CocoaPods consumers to also get a copy of the framework.

@nitaliano
Copy link
Owner

@orta what will happen if I pull in the native mapbox ios sdk thru my podfile?

@orta
Copy link
Contributor Author

orta commented Feb 6, 2019

Hrm, good Q, let's see how that works - might be able to close this

@orta
Copy link
Contributor Author

orta commented Feb 6, 2019

Looking good, I get framework conflicts. Perhaps this PR can be switched to documentation on the README.

@nitaliano
Copy link
Owner

@orta thanks for checking it out, I with updating the documentation on this PR and we can get it merged in.

@orta
Copy link
Contributor Author

orta commented Mar 12, 2019

Happy to see where this goes instead: #1533

@orta orta closed this Mar 12, 2019
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.

2 participants