-
Notifications
You must be signed in to change notification settings - Fork 10.5k
build: switch to CMake for XCTest #17180
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
Please test with following PR: @swift-ci please test |
Build failed |
Build failed |
utils/build-script-impl
Outdated
-DENABLE_TESTING=YES | ||
) | ||
;; | ||
easc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo, I think? s/easc/esac/ -- maybe that's why the build failed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh, right-e-o.
@swift-ci please test |
Build failed |
Build failed |
Now it looks like the SwiftPM tests Linux are failing due to not being able to import XCTest... I wonder if they expect the build products to be someplace they're not, sort of like what happened in swiftlang/swift-corelibs-xctest#215 ? |
@modocache - yeah, that would be my guess, seems that the layout might be different :-( |
Ah, I see, the cmake builds have placed the swift content under a |
Please test with following PR: @swift-ci please test Linux platform |
Build failed |
CC: @aciidb0mb3r - any idea why s-p-m fails to find xctest? |
@swift-ci please test Linux platform |
1 similar comment
@swift-ci please test Linux platform |
Build failed |
@swift-ci please test Linux platform |
Build failed |
@swift-ci please test |
Build failed |
Please test with following pull request: @swift-ci Please test Linux platform |
Build failed |
@swift-ci please test Linux platform |
Build failed |
@swift-ci please test Linux platform |
Build failed |
Now that XCTest has a CMake build system, use that for the non-Darwin builds. This sets the stage for building Foundation with CMake.
@swift-ci please test |
Build failed |
Build failed |
@swift-ci please test Linux platform |
Build failed |
I was able to build this on linux locally. I think we may have a dependency problem (not sure). Lets try a clean build to see if my intuition is right. |
To be clear, if I am correct that this is a dependency problem, we should eliminate that dependency problem first before this merges. |
@swift-ci clean test linux platform |
Build failed |
@swift-ci test linux platform |
1 similar comment
@swift-ci test linux platform |
Log was eliminating. I am re-starting the incremental build to see if we can reproduce on the bots again. |
Build failed |
@swift-ci please test |
Build failed |
@swift-ci please test linux platform |
Build failed |
@swift-ci please test linux platform |
Build failed |
Please test with following PR: @swift-ci please test |
@compnerd congrats! |
Looks like XCTest is no longer getting rebuilt when the standard library changes. What can we do about that? |
Now that XCTest has a CMake build system, use that for the non-Darwin
builds. This sets the stage for building Foundation with CMake.
Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.
Resolves SR-NNNN.