Skip to content

Add xdgTestHelper into the TestFoundation bundle as an auxiliaryExecutable #1564

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
merged 1 commit into from
May 29, 2018

Conversation

spevans
Copy link
Contributor

@spevans spevans commented May 26, 2018

  • For Linux, set the output executable from the swiftc to go into
    the TestFoundation application directory.

  • For DarwinCompatibiltyTests, build xdgTestHelper as a command
    line tool then add a build phase to copy it to the 'Executables'
    destination.

  • For TestFoundation.app, build xdgTestHelper as a cocoa .app so
    that Xcode doesnt statically link it to the Foundation from the
    Swift toolchain, then copy the binary.

This is a followup to #1562 which made xdgTestHelper into a standalone single .swift file.

…table.

- For Linux, set the output executable from the swiftc to go into
  the TestFoundation application directory.

- For DarwinCompatibiltyTests, build xdgTestHelper as a command
  line tool then add a build phase to copy it to the 'Executables'
  destination.

- For TestFoundation.app, build xdgTestHelper as a cocoa .app so
  that Xcode doesnt statically link it to the Foundation from the
  Swift toolchain, then copy the binary.
@spevans
Copy link
Contributor Author

spevans commented May 26, 2018

@swift-ci please test

@spevans spevans requested a review from millenomi May 28, 2018 15:16
@spevans
Copy link
Contributor Author

spevans commented May 29, 2018

@swift-ci please test and merge

@swift-ci swift-ci merged commit 920035a into swiftlang:master May 29, 2018
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