mapbox/mapbox-navigation-ios

[Bug]: Installation fails in React Native Expo Managed app

Opened this issue ยท 2 comments

Mapbox Navigation SDK version

2.16

Steps to reproduce

Hello!

I'm trying to build a Native Module in my React Native Expo-managed app, to wrap the Mapbox Navigation iOS and Android SDKs.

The issue arises when I try to install the app with the MapboxNavigation dependency on it, regardless if I do it via the Expo CLI or via Xcode.

Error during compile time: Multiple commands produce '/Users/MyUser/Library/Developer/Xcode/DerivedData/proj-identifier/Build/Products/Debug-iphonesimulator/mapboxpoc.app/Assets.car'.

I've been investigating the issue and got the installation working by following approach 3 in this blog. This approach requires me to mess directly with the Build phases of the xcode project, which is not ideal since the whole idea behind Expo is to avoid manually editing the iOS and Android folders. In fact, the iOS project gets regenerated on every build (it is even ignored from version control! ).

Expo uses Cocoapods to fetch Pods. Here's how I've listed the dependency in my module's podspec:

  s.dependency 'MapboxNavigation', '~> 2.16'

I know this is the iOS repo, that you don't support React Native officially, and perhaps my issue will be dismissed ๐Ÿ™‚ .. But I would really appreciate if you could at least take a quick look at it. Perhaps it's a really easy fix for you, and you would be helping plenty of people who are trying to adopt Expo in the React Native ecosystem and want to use your library.

Thanks a lot in advance!

Expected behavior

The build process should be successful without any errors ๐Ÿคž

Actual behavior

It fails with error Multiple commands produce '/Users/MyUser/Library/Developer/Xcode/DerivedData/proj-identifier/Build/Products/Debug-iphonesimulator/mapboxpoc.app/Assets.car' ๐Ÿ™

image

Is this a one-time issue or a repeatable issue?

repeatable

Hi, @santitopo!
I believe there is a PR which addresses this issue, but we didn't have a chance to verify and merge it yet :(

Hey @Udumft ! Thanks for the quick response!! Really appreciate it ๐Ÿ™Œ Hadn't seen that ticket, but it does seems to be the same issue ๐Ÿ‘

May I ask if there's an expected timeline for it's merge and release?

Thanks again!