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

No modules on library targets; imports Metal #70

Merged
merged 1 commit into from
Sep 23, 2014

Conversation

jmoody
Copy link
Contributor

@jmoody jmoody commented Sep 21, 2014

Xcode 5.1.1 users are reporting problems when calabash.framework is built with Xcode 6*.

Modules are required with Xcode 6 for dylibs.

The calabash.framework and frank library do not need to use modules, so we should turn them off.

Xcode 5.1.1 users are reporting problems when calabash.framework
is built with Xcode 6*.

Modules are _required_ with Xcode 6 for dylibs.

The calabash.framework and frank library do not need to use
modules, so we should turn them off.
* calabash/calabash-ios#519
* card-io/card.io-iOS-SDK#65
* CocoaPods/CocoaPods#2457
@jmoody jmoody added this to the 0.10.2 milestone Sep 21, 2014
@jmoody
Copy link
Contributor Author

jmoody commented Sep 21, 2014

@krukow Have a look and let me know what you think. This is how I tested.

[briar-ios-example/Briar] $ be calabash-ios download
[briar-ios-example/Briar] $ DEVELOPER_DIR=/Xcode/5.1.1/Xcode.app/Contents/Developer ./xtc-prepare.sh
> ld: framework not found Metal for architecture armv7
# Disable ObjC Modules in calabash-ios-server
[briar-ios-example/Briar] $ be briar install calabash-server
[briar-ios-example/Briar] $ ./xtc-prepare.sh
> Archive succeed.

Jenkins CI has been failing, but I thought it was failing because of an unrelated (and pesky) code-signing problem.

jmoody added a commit that referenced this pull request Sep 23, 2014
…ry-targets

No modules on library targets; imports Metal
@jmoody jmoody merged commit 0c6a208 into develop Sep 23, 2014
@jmoody jmoody deleted the feature/disable-modules-on-library-targets branch September 23, 2014 09:00
@jmoody jmoody mentioned this pull request Sep 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants