-
Notifications
You must be signed in to change notification settings - Fork 1.2k
build: repair the build on 14.04 #1747
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
Conversation
@swift-ci please test |
@swift-ci please test macOS platform |
f243471
to
32c8866
Compare
@swift-ci please test |
@swift-ci test linux platform |
@swift-ci test |
CMake 3.4.3 does not support the <files>... <directory> pattern for copy_if_different. Adjust for that.
32c8866
to
edfd25a
Compare
@swift-ci please test |
Looking at the Ubuntu14.04 build host logs they show the cmake version as 3.4.3 but https://github.com/apple/swift/blob/master/docs/Ubuntu14.md says that 3.5.2 should be installed. Maybe we should just get the 14.04/16.04 hosts updated to 3.10 and then set a minimum version check in Swift's |
@swift-ci please test |
@spevans A few things:
|
I didn't realise there would be those issues. But could we at least get the Ubuntu14 CI hosts installed with 3.5.2 as listed in the docs? - I have it on my Ubuntu14 test host and the full toolchain builds and tests ok without the need for the fix in this PR. |
@spevans two things:
Doing 1 would require a post to swift-dev and potential discussion I think. |
Btw, it looks like the macOS changes are related to some clang changes: https://ci.swift.org/job/oss-swift-incremental-RA-osx/5513/ and not to this PR. I am going to merge this to unbreak the 14.04 build. |
CMake 3.4.3 does not support the ... pattern for
copy_if_different. Adjust for that.