-
-
Notifications
You must be signed in to change notification settings - Fork 839
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
7.0.10 crashing on start #635
Comments
Published 7.0.10 now. Can you try again? |
The same issue. It is version 7.0.10 that is crashing the app immediately after it is launched. |
Likely root cause at
mapbox/mapbox-gl-native-ios#131
We’ll need to update download path and republish. I’ll do it tonight.
…On Wed, 22 Jan 2020 at 17:39, Karolis Januškas ***@***.***> wrote:
The same issue. It is version 7.0.10 that is crashing the app immediately
after it is launched.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/react-native-mapbox-gl/maps/issues/635?email_source=notifications&email_token=ABLAPWZWXJSCT7Z6AOKXZN3Q7BZEJA5CNFSM4KKINBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJUIFAY#issuecomment-577274499>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLAPW2NMIALBQY3O6O6NIDQ7BZEJANCNFSM4KKINBMQ>
.
|
Ok, that was a bigger issue than expected. I think I fixed it. Can you try https://github.com/react-native-mapbox-gl/maps/pull/637 ? To install it, do |
Merged this into master, you can install by |
I'm also having the crash-on-start issue using Steps to reproduce:
|
@makozlo-airship Is this still happening for you? Or figure it out. |
Make sure you unlink existing FW and use pods to install the new version @makozlo-airship Please try that and get back to me if the issue persists. |
@kristfal I did a clean install...I think I might be confused about installation. I'm using RN version |
@makozlo-airship did you also |
I have the same issue (app crashing on start) with the following error: dyld: Library not loaded: @rpath/MapboxMobileEvents.framework/MapboxMobileEvents
Referenced from: /Users/cglacet/Library/Developer/CoreSimulator/Devices/7CBB4959-C16B-4A17-BA68-7BBE7A046BBE/data/Containers/Bundle/Application/724057ED-70A3-49DB-A407-7014EA21EA80/AppName.app/Frameworks/Mapbox.framework/Mapbox
Reason: image not found My dependencies are: "@react-native-mapbox-gl/maps": "^7.0.10",
"@mapbox/geo-viewport": "^0.2.2",
"@mapbox/mapbox-sdk": "^0.6.0",
"@mapbox/polyline": "^1.0.0", My config is the following: $ npx react-native info
System:
OS: macOS Mojave 10.14.6
CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Memory: 88.09 MB / 16.00 GB
Shell: 5.7.1 - /usr/local/bin/zsh
Binaries:
Node: 12.14.1 - /usr/local/bin/node
Yarn: 1.3.2 - /usr/local/bin/yarn
npm: 6.13.4 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
IDEs:
Android Studio: 3.5 AI-191.8026.42.35.5977832
Xcode: 11.3/11C29 - /usr/bin/xcodebuild
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.5 => 0.61.5 I ran both |
Please update to 7.2.0 and redo installation. Does it work now?
…On Fri, 24 Jan 2020 at 10:10, Christian glacet ***@***.***> wrote:
I have the same issue (app crashing on start) with the following error:
dyld: Library not loaded: @rpath/MapboxMobileEvents.framework/MapboxMobileEvents
Referenced from: /Users/cglacet/Library/Developer/CoreSimulator/Devices/7CBB4959-C16B-4A17-BA68-7BBE7A046BBE/data/Containers/Bundle/Application/724057ED-70A3-49DB-A407-7014EA21EA80/AppName.app/Frameworks/Mapbox.framework/Mapbox
Reason: image not found
My dependencies are:
***@***.***/maps": ***@***.***/geo-viewport": ***@***.***/mapbox-sdk": ***@***.***/polyline": "^1.0.0",
My config is the following:
$ npx react-native info
System:
OS: macOS Mojave 10.14.6
CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Memory: 88.09 MB / 16.00 GB
Shell: 5.7.1 - /usr/local/bin/zsh
Binaries:
Node: 12.14.1 - /usr/local/bin/node
Yarn: 1.3.2 - /usr/local/bin/yarn
npm: 6.13.4 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
IDEs:
Android Studio: 3.5 AI-191.8026.42.35.5977832
Xcode: 11.3/11C29 - /usr/bin/xcodebuild
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.5 => 0.61.5
I ran both npm install and pod install without any error showing up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/react-native-mapbox-gl/maps/issues/635?email_source=notifications&email_token=ABLAPW6YNLX43KGJPGJLLFLQ7KWBTA5CNFSM4KKINBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ2FLNA#issuecomment-578049460>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLAPW6YBP2WI2G5MKSEAQDQ7KWBTANCNFSM4KKINBMQ>
.
|
Currently trying it, so far I've ran: $ yarn add https://github.com/react-native-mapbox-gl/maps.git#update-ios-framework
$ cd ios
$ pod install
...
Installing Mapbox-iOS-SDK (5.6.1)
Installing MapboxMobileEvents (0.10.2)
Installing react-native-mapbox-gl 7.2.0 (was 7.0.10)
... For now I still have the same error message when running from xcode. |
@cglacet can you try to run the example? Eg:
If this work, and your project doesn't, it is likely because the old Mapbox.framework is still referenced in your project. You need to make sure you properly unlink it before you add it again. |
I tried to install the example, but
To be a bit more specific:
|
I just tried running the example project and
|
same issue i am also facing in "@react-native-mapbox-gl/maps": "^7.2.0", app crashes on start
|
adding use_frameworks!, solves crash on launch but it cause issue for other pods |
Good to hear it doesn’t crash.
Unfortunately Mapbox native iOS SDK now require use_frameworks, so there is
no way around it with pods afaik.
…On Fri, 24 Jan 2020 at 19:02, waris117 ***@***.***> wrote:
adding use_frameworks!, solves crash on launch but it cause issue for
other pods
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/react-native-mapbox-gl/maps/issues/635?email_source=notifications&email_token=ABLAPW4V33Q4V6S742E2PJDQ7MUJDA5CNFSM4KKINBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ3TANI#issuecomment-578236469>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLAPW43H2ALKGR5SHTS2ALQ7MUJDANCNFSM4KKINBMQ>
.
|
Getting the same thing after upgrading to 7.2.0: /Frameworks/Mapbox.framework/Mapbox The use_frameworks! situation is very problematic for many applications including ours. We cannot use many of the most popular libraries from Google including Crashlytics, Analytics, etc. etc. when we have use_frameworks! in the podfile. If this is required to use the new pod version of Mapbox SDK for iOS than I'm really concerned about how to make this work properly moving forward. |
@systemlevel yeh, I totally get that. While it is an option, I'm not super stoked about going back to downloading the precompiled framework. The use_frameworks! requirement is a choice made by Mapbox upstream (as I see you noticed), and I hope they can provide a good pod-based experience that we can follow. I'd say we should do the following:
|
@kristfal Thank you! And thank you for the efforts in bringing the cocoapods over. That sounds like a great plan moving forward. |
@kristfal I was having issues with the |
No news from Mapbox. I’d Ideally like them to respond to mapbox/mapbox-gl-native-ios#154 before we make a move. I suggest you subscribe to that ticket if you want to follow this. If I don’t hear anything by next week, I’ll get the workaround in place. |
In the meantime, how could we (as end users) revert to previous version of the iOS SDK? Is it sufficient to rollback to a different version of react-native-mapbox-gl/maps? Which one? It looks like version 7.0.9 is fine (it installs iOS SDK version 5.5.0) |
The interim solution if you can't use |
@kristfal when I try running
|
Forgot to add the @ before the install path. Updated code should work. |
@kristfal Just tried again, not working. Will your change take a few minutes to become available? |
No, there is no delay. This works on my end:
If you get a different result, check your network connection, cache etc. |
Weird. Just ran it again and it installed fine. Thanks! |
+1 :( |
@kristfal I tried the new version(7.0.9) on android and i was getting this crash
react-native version: 0.61.5 |
@afomic I had to go back to 7.0.8 to get things working on android. I forgot what the issue that 7.0.9 caused on Android, but there was something. I think it was hermes related. |
@kristfal I've been following this thread for a while (as well as issue #154), and haven't seen any updates. I know you mentioned you would implement a workaround if you didn't hear back from Mapbox after a week or so, do you still have plans to do this? |
I tried to go for 7.0.10. I thought it might work, because the first mention of the new SDK in iOS was for 7.2.0, here: Alas, 7.0.9 is the max version I can get to work without use_frameworks. |
I forked this branch at the point where the Android SDK was updated to 8.6.1, and then reverted the iOS SDK back to 5.5.0. In case you want to install that too: With the code above, I was able to get the app to start on both iOS and Android. |
This is still happening, has anyone figured out a fix? |
Looks like there was some activity from Mapbox here: mapbox/mapbox-gl-native-ios#154. @julianrex is indicating no short-term action will be taken on use_frameworks issue. Wondering what the community's thoughts are on a path forward? There continues to be many valuable contributions to this library we'd love to test and include but we have to hold back for the time being. |
|
@mfazekas got it fixed by reinstalling the node module, but now Im also getting the |
I have same error with 7.2.0 version, have you found a solution please? |
Any news on this guys? 7.2.0 is still crashing on startup |
Having the same issue with |
I reverted to 7.0.9 to get it work on both platforms. |
To everyone having issues with Android, could you please try to reproduce on |
Any updates? It still crashes on start on real iPhone. |
There is PR #714 which offers a workaround for |
on real devices? |
@kk12837 yes. |
I'm closing this as there are multiple issues here. |
Describe the bug
app crashes on start
To Reproduce
install version 7.0.10 and try to start the app
Expected behavior
does not crash
Versions (please complete the following information):
The text was updated successfully, but these errors were encountered: