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

Explicitly set code signing #78

Merged
merged 2 commits into from
Sep 2, 2015
Merged

Explicitly set code signing #78

merged 2 commits into from
Sep 2, 2015

Conversation

gfontenot
Copy link
Member

Not having this set was causing intermittent build failures when building with Carthage.

Similar to ReactiveCocoa/ReactiveCocoa#2292

I'm not sure that this is the best solution to this, but I'm not sure it's a bad solution either. If someone has more knowledge behind why this is happening, I'm all ears.

Not having this set was causing build failures when building with
Carthage.

Similar to ReactiveCocoa/ReactiveCocoa#2292
@JaviSoto
Copy link
Contributor

JaviSoto commented Sep 2, 2015

👍 I think this is correct.

@@ -716,6 +718,7 @@
isa = XCBuildConfiguration;
buildSettings = {
CLANG_ENABLE_MODULES = YES;
CODE_SIGN_IDENTITY = "iPhone Developer";
"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";
Copy link
Member

Choose a reason for hiding this comment

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

Probably want to delete these though

@gfontenot
Copy link
Member Author

@neilpa good catch, fixed those.

@neilpa
Copy link
Member

neilpa commented Sep 2, 2015

Seeing as I made the same change for RAC 👍 as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants