Skip to content
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

Include PLATFORM_NAME in bundle identifier #751

Merged
merged 1 commit into from
Sep 4, 2019

Conversation

nchase
Copy link
Contributor

@nchase nchase commented Sep 3, 2019

As of ~5 days ago, App Store submissions will fail with the ITMS-90806 publishing error code if there are multiple bundles with the same identifier. This is a problem for apps that e.g. include an iOS version alongside a watchOS version.

This change just appends the PLATFORM_NAME to the bundle identifier to get around the issue.

Here's a thread from Apple's message boards about the issue: https://forums.developer.apple.com/thread/122048

As of ~5 days ago, App Store submissions will fail with the ITMS-90806 publishing error code if there are multiple bundles with the same identifier. This is a problem for apps that e.g. include an iOS version alongside a watchOS version.

This change just appends the `PLATFORM_NAME` to the bundle identifier to get around the issue.

Here's a thread from Apple's message boards about the issue: https://forums.developer.apple.com/thread/122048
@apollo-cla
Copy link

@nchase: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@designatednerd
Copy link
Contributor

@nchase There's a thread on CocoaPods' Github about this - Are you using CP to integrate or another method? If it's only CocoaPods having this issue, I'm inclined to recommend this be handled with one of the post_install workarounds that fixes this problem for every framework rather than just for ours.

If you're not using CocoaPods, what are you using?

@designatednerd designatednerd added the dependency-management Issues with CocoaPods, Carthage, or SPM integration label Sep 4, 2019
Copy link
Contributor

@designatednerd designatednerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buh, looking into Alamofire's handling it appears this is an issue with Carthage too. Thanks so much, 🍎🙄

@designatednerd designatednerd merged commit 7e3fd3e into apollographql:master Sep 4, 2019
@designatednerd designatednerd added this to the 0.15.1 milestone Sep 4, 2019
@nchase nchase deleted the patch-1 branch September 4, 2019 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency-management Issues with CocoaPods, Carthage, or SPM integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants