Skip to content

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