Ziggeo / ReactNativeSDK

React Native SDK
Apache License 2.0
7 stars 6 forks source link

IOS can't enabled bitcode #100

Closed sam-houle1 closed 1 year ago

sam-houle1 commented 2 years ago

I can't activate ENABLED_BITCODE in my build settings, if I do my app won't start here's the error:

'/Users/samuelhoule/Web/MyProject/project/ios/Pods/iOS-Client-SDK/Output/SelfieSegmentation.framework/SelfieSegmentation' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. file '/Users/samuelhoule/Web/MyProject/project/ios/Pods/iOS-Client-SDK/Output/SelfieSegmentation.framework/SelfieSegmentation' for architecture arm64

I need to activate bitcode to upload sourcemaps to sentry.

sam-houle1 commented 2 years ago

@3akat Hi is it possible to look into this issue ?

3akat commented 2 years ago

Hey @sam-houle1, we are working in sprints and because we are already mid sprint we are not going to be able to add this to the current sprint. We will be adding this to the next sprint, which means 3-4 weeks time. Of course as we see it as a bug, we will add priority so that we start on it as soon as it is possible.

sam-houle1 commented 2 years ago

@3akat okok but this is a big problem right now, pretty sure everybody using this package can't generate sourcemaps for firebase or sentry. I don't think I can wait 3-4 weeks without logging on my app. Unfortunately, if that's really the decision, I'll might have to delete this package entirely and use a different service. Since this package have a lot of problem, if every time there's a major bug, I have to wait 3-4 weeks to have a fix, I don't think my app is going to survive.

3akat commented 2 years ago

@sam-houle1 that's true, we will do our best to fix this asap

sam-houle1 commented 2 years ago

@3akat thanks I appreciate

sam-houle1 commented 2 years ago

Hi @3akat is it possible to have an ETA on when is this going to be fix ?

3akat commented 2 years ago

Hey @sam-houle1. Just a few hours ago we had internal discussion regarding this issue - we can't fix it fast, but we will create a separate release which excludes the feature-cause. Hope to see it until the end of the week.

sam-houle1 commented 2 years ago

@3akat alright perfect thanks for the heads up !

sam-houle1 commented 2 years ago

hi @3akat, is this fix ?

Bane-D commented 2 years ago

Hi Sam, thank you for your patience so far.

My colleagues would know more, however as I saw your question, I thought to give you the info that I know.

The update that addresses this will be made soon, it was not pushed out yet.

Once it is available one of my colleagues will follow up with you to let you know it is available for you to test it. :)

Bane-D commented 2 years ago

Hi Sam,

The code has been updated and should work right for you now.

If you want to use our SDK with standard features (without the background blurring) then for iOS you would:

  1. Create podfile within your project
  2. add:
    pod 'iOS-Client-SDK', :git => 'https://github.com/Ziggeo/iOS-Client-SDK.git'

    If you want to use SDK with the background blurring then for iOS you would:

  3. Create podfile within your project
  4. add:
    pod 'iOS-Client-SDK', :git => 'https://github.com/Ziggeo/iOS-Client-SDK.git', :branch => 'blurring'

Do let us know how it goes.

sam-houle1 commented 2 years ago

@Bane-D Do I have to upgrade my ziggeo nom package ? Currently on 1.6.11, So basicly I didn't have the blurring option, so I had nothing to do except enabled bitcode, and I still have the same error.

Bane-D commented 2 years ago

Hey Sam, I will check with my colleague, however you will likely need to clean the build first so:

$ cd ios
$ rm -rf build
$ xcodebuild clean

then try to build it (with the podfile updated - if needed). It should then pull the latest iOS Client SDK version for you.

katolykdev commented 2 years ago

@sam-houle1
If your app needs to set the ENABLED_BITCODE option to Yes, please add the below code to Pod file. pod 'iOS-Client-SDK', :git => 'https://github.com/Ziggeo/iOS-Client-SDK.git'

And reinstall pod. pod install

That sdk was updated several days ago. Please recheck.

sam-houle1 commented 2 years ago

@katolykdev @Bane-D Hi, I was able to launch the app on my device, but I still have the same error if I try to use the simulator.

katolykdev commented 2 years ago

@sam-houle1 I will check on simulator.

katolykdev commented 2 years ago

@sam-houle1 I have checked on simulator, it is working. Could you send me the screenshot of the error?

sam-houle1 commented 2 years ago
Screen Shot 2022-06-01 at 12 52 27 PM

@katolykdev It's the exact same error as before.

katolykdev commented 2 years ago

@sam-houle1 Did you install the latest version of iOS-Client-SDK? Could you tell me the version you installed?

sam-houle1 commented 2 years ago

@katolykdev i'm on 1.1.37

katolykdev commented 2 years ago

@sam-houle1 Could you send me the project including package file and Pod file. The src folder is not necessary.

sam-houle1 commented 1 year ago

@katolykdev Hi, I was wondering if this is something you guys could reproduce, I would rather not have to send my private code base.

katolykdev commented 1 year ago

@sam-houle1 Hello, I updated iOS SDK and I will release in 1 week. User can enable bitcode on this version.