Use swiftc for linking.

[swiftix]

Using swiftc for linking is more reliable. It also properly handles auto-linking.

[swiftix]

@swift-ci Please test

[swiftix]

Hmm. Looks like the swift-ci trick is not supported for this project ;-(

[swiftix]

@parkera OK to merge?

[modocache]

@swiftix Swift CI should have been enabled 13 days ago: #45 (comment). swiftlang/swift#1316 adds support for tests.

[modocache]

This change looks fine. I'm curious, how did you test this change locally? Did you try running the tests?

./build_script.py \
    --swiftc="/swift/usr/bin/swiftc" \
    --build-dir="/tmp/XCTest_build" \
    --swift-build-dir="/swift/usr" \
    --arch="x86_64" \
    --test

[swiftix]

I tested it as a part of a Swift build process, which included testing this project as well.

[swiftix]

@modocache As for Swift CI. It has been started by my comment. But it fails instantly because of some git issues or something like that. So essentially Swift CI for this project is broken at the moment, which prevents it from testing new PRs.

[swiftix]

@modocache More details are available if you look at the Swift CI logs.

[modocache]

@swiftix Thanks! Just to clarify: the tests for this project aren't run as part of any Swift utils/build-script invocation--at least, they won't be until swiftlang/swift#1316 is merged.

[swiftix]

I think the change is fine. But I normally do not touch swift-corelibs-xctest. This is why I want someone more familiar with this project to click the Merge button.

[swiftix]

@modocache I have my our build-script preset that I used for that.

[swiftix]

I also tested using the command you provided above. No test failures.

[modocache]

@swiftix Awesome!! Thanks so much. If I had commit access, I'd merge this in a heartbeat. @mike-ferris-apple is the main steward of this project but is away for the week. It would be great if @parkera or @gribozavr could merge. 🙇