Skip to content

[CMake] Install swiftinterface files alongside swiftmodules #19263

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

jrose-apple
Copy link
Contributor

This may not be their final destination, but at least this will get them into built toolchains, which means it'll be easier to do integration testing once all the other pieces are in place.

rdar://problem/43823937

These should install the swiftmodule and swiftdoc files too.
This may not be their final destination, but at least this will
get them into built toolchains, which means it'll be easier to
do integration testing once all the other pieces are in place.

rdar://problem/43823937
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@jrose-apple
Copy link
Contributor Author

@swift-ci Please build toolchain

@swift-ci
Copy link
Contributor

Linux Toolchain (Ubuntu 16.04)
Download Toolchain
Git Sha - bc80fdd

Install command
tar zxf swift-PR-19263-59-ubuntu16.04.tar.gz
More info

@swift-ci
Copy link
Contributor

macOS Toolchain
Download Toolchain
Git Sha - bc80fdd

Install command
tar -zxf swift-PR-19263-71-osx.tar.gz --directory ~/

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - bc80fdd

@jrose-apple
Copy link
Contributor Author

@swift-ci Please test Linux

Copy link
Contributor

@Rostepher Rostepher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! 👍

@jrose-apple jrose-apple merged commit bdb3a45 into swiftlang:master Sep 13, 2018
@jrose-apple jrose-apple deleted the install-installer-installest branch September 13, 2018 17:51
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