Skip to content

if building Swift overlay, install into Swift toolchain dir structure #49

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
Feb 15, 2016

Conversation

dgrove-oss
Copy link
Contributor

Simplify the build/install logic for the Swift overlay portion of
libdispatch to assume that if the Swift overlay is being built
then the intended install target is a Swift toolchain.

In conjunction with the change to swift/utils/build-script-impl
in swiftlang/swift#1212, this change enables Swift programs compiled using
the resulting built toolchain to successfully import
Dispatch with only -Xcc -fblocks as extra arguments to swiftc
(no longer need to also specify -I).

Simplify the build/install logic for the Swift overlay portion of
libdispatch to assume that if the Swift overlay is being built
then the intended install target is a Swift toolchain.

In conjunction with the change to swift/utils/build-script-impl
in PR#1212, this change enables Swift programs compiled using
the resulting built toolchain to successfully import
Dispatch with only -Xcc -fblocks as extra arguments to swiftc
(no longer need to also specify -I<path-to-dispatch-module>).
MadCoder added a commit that referenced this pull request Feb 15, 2016
if building Swift overlay, install into Swift toolchain dir structure
@MadCoder MadCoder merged commit f1d58d1 into swiftlang:master Feb 15, 2016
@dgrove-oss dgrove-oss deleted the swift-overlay-cp2-squashed branch February 16, 2016 13:22
das pushed a commit that referenced this pull request Jun 14, 2016
if building Swift overlay, install into Swift toolchain dir structure

Signed-off-by: Daniel A. Steffen <[email protected]>
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