launchdarkly / ios-client-sdk

LaunchDarkly Client-side SDK for iOS (Swift and Obj-C)
https://docs.launchdarkly.com/sdk/client-side/ios
Other
69 stars 84 forks source link

Carthage Based Build Fails App Store Validation #217

Closed spr closed 4 years ago

spr commented 4 years ago

Is this a support request? No

Describe the bug If you use LaunchDarkly 5.0.1 as a dependency to your project via Carthage, and submit your .xcarchive file to the App Store for validation, you receive the following validation failures:

To reproduce Steps to reproduce the behavior.

  1. Create Project
  2. Add LaunchDarkly via Carthage
  3. Generate an archive
  4. "Validate App" in the Xcode organizer

Expected behavior A clear and concise description of what you expected to happen. App should pass validation

Logs If applicable, add any log output related to your problem.

SDK version The version of this SDK that you are using. 5.0.1

Language version, developer tools Xcode 11.6

OS/platform macOS 10.15.5

Additional context Add any other context about the problem here.

The LaunchDarkly framework is embedding the LDSwiftEventSource framework. It shouldn't do that, LDSwiftEventSource should be added directly to the top level project.

gwhelanLD commented 4 years ago

Hi @spr,

Thanks for also testing this in Carthage. Like https://github.com/launchdarkly/ios-client-sdk/issues/216, we will also prioritize including a fix for this in 5.1 to resolve the app store submission rejection error. We appreciate your detailed reports!

Thanks, @gwhelanLD

gwhelanLD commented 4 years ago

Hi @spr,

We've just released 5.1.0 which should resolve this issue. Please give it a try and let us know if you run into any issues.

Thanks! @gwhelanLD

spr commented 4 years ago

Passes validation now, thank you!