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

Allow to be consumed without use_frameworks! #714

Merged
merged 1 commit into from
Mar 11, 2020

Conversation

mfazekas
Copy link
Contributor

@mfazekas mfazekas commented Mar 10, 2020

This is a possible workaround for use_frameworks!

If you don't want use_frameworks! in your podfile use the pod 'NoUseFrameworks-MapboxMobileEvents' ... line bellow:

    pod 'NoUseFrameworks-MapboxMobileEvents',  :podspec => '../node_modules/@react-native-mapbox-gl/maps/ios/NoUseFrameworks-MapboxMobileEvents/NoUseFrameworks-MapboxMobileEvents.podspec.json'
  # use_frameworks!

Note: you can test this PR with this in your packages.json

"@react-native-mapbox-gl/maps": "mfazekas/maps-1#allow-no-use-frameworks"

See mapbox/mapbox-gl-native-ios#154

@mfazekas mfazekas changed the title Allow to be consumed without use_frameworks! RFC: Allow to be consumed without use_frameworks! Mar 10, 2020
@jeremybarbet
Copy link
Contributor

Tested it and it worked on my side. That's really great, I ship it to production :D

@systemlevel
Copy link
Contributor

systemlevel commented Mar 11, 2020

Hi @mfazekas, I also ran this through on our end. This appears to be working. App also ran successfully.

Screen Shot 2020-03-10 at 10 23 05 PM

@systemlevel
Copy link
Contributor

@mfazekas Thank you for your work on this PR. Very much appreciated!!

@mfazekas
Copy link
Contributor Author

mfazekas commented Mar 11, 2020

Probably i'd rename the podspec to

../node_modules/@react-native-mapbox-gl/maps/ios/NoUseFrameworks-MapboxMobileEvents/NoUseFrameworks-MapboxMobileEvents.podspec.json

@systemlevel
Copy link
Contributor

@mfazekas that sounds good to me. Good and clear.

@mfazekas mfazekas force-pushed the allow-no-use-frameworks branch from ec0eba8 to f2a1fd6 Compare March 11, 2020 07:27
@mfazekas mfazekas changed the title RFC: Allow to be consumed without use_frameworks! Allow to be consumed without use_frameworks! Mar 11, 2020
@mfazekas mfazekas merged commit 4be9f1d into rnmapbox:master Mar 11, 2020
@tanguy64
Copy link

Tested and working ! Thanks a lot @mfazekas !

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.

4 participants