Skip to content

Preemptively add XCTest.xcworkspace [AsyncXCTest 2/6] #63

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

Merged

Conversation

modocache
Copy link
Contributor

This is the second of six pull requests necessary to introduce asynchronous testing to swift-corelibs-xctest. @modocache will merge this once the CI passes.

#43 will add a dependency between swift-corelibs-xctest and swift-corelibs-foundation. On OS X, this will also necessitate an Xcode workspace that references both XCTest and Foundation .xcodeproj's.

To prevent CI from breaking when #43 is landed, preemptively add an Xcode workspace. Before landing #43, the Swift build script will be modified to build XCTest.xcworkspace, not XCTest.xcodeproj.

@modocache
Copy link
Contributor Author

Here's how the build script will need to change: modocache/swift@71f20f6

Notice that it builds the XCTest workspace.

swiftlang#43 will add a dependency
between swift-corelibs-xctest and swift-corelibs-foundation. On OS X, this
will also necessitate an Xcode workspace that references both XCTest
and Foundation .xcodeproj's.

To prevent CI from breaking when swiftlang#43 is landed, preemptively add an Xcode
workspace. Before landing swiftlang#43, the Swift build script will be modified to
build XCTest.xcworkspace, not XCTest.xcodeproj.
@modocache modocache force-pushed the xctestexpectation-workspace-migration branch from 889134b to a24d5fc Compare March 5, 2016 21:20
@briancroom
Copy link
Contributor

Cool!

There are a lot of steps to this dance.

@modocache modocache changed the title Preemptively add XCTest.xcworkspace Preemptively add XCTest.xcworkspace [AsyncXCTest 2/5] Mar 7, 2016
@modocache modocache self-assigned this Mar 7, 2016
@modocache modocache changed the title Preemptively add XCTest.xcworkspace [AsyncXCTest 2/5] Preemptively add XCTest.xcworkspace [AsyncXCTest 2/6] Mar 7, 2016
@mike-ferris
Copy link

@swift-ci please test

@mike-ferris
Copy link

@swift-ci Please test

modocache added a commit that referenced this pull request Mar 7, 2016
…ration

Preemptively add XCTest.xcworkspace [AsyncXCTest 2/6]
@modocache modocache merged commit 50d3823 into swiftlang:master Mar 7, 2016
@modocache modocache deleted the xctestexpectation-workspace-migration branch March 7, 2016 22:04
@modocache
Copy link
Contributor Author

The OS X failure is expected. I think @shahmishal is working on getting xcodebuild working with the latest Swift toolchain (which is necessary for #file and #line identifiers to compile).

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.

3 participants