Skip to content

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

Merged
merged 1 commit into from
Sep 21, 2018
Merged

Conversation

compnerd
Copy link
Member

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.

@compnerd
Copy link
Member Author

compnerd commented Jun 13, 2018

CC: @briancroom @modocache @gottesmm

@compnerd
Copy link
Member Author

Please test with following PR:
swiftlang/swift-corelibs-xctest#215

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 06eee5287e34038646de4b1c90d209b3e55029df

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 06eee5287e34038646de4b1c90d209b3e55029df

-DENABLE_TESTING=YES
)
;;
easc
Copy link
Contributor

@modocache modocache Jun 13, 2018

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?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ugh, right-e-o.

@compnerd
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 06eee5287e34038646de4b1c90d209b3e55029df

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 184c41e9f9805c0d45cadf5eb3a433c14f7dc105

@modocache
Copy link
Contributor

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 ?

@compnerd
Copy link
Member Author

@modocache - yeah, that would be my guess, seems that the layout might be different :-(

@compnerd
Copy link
Member Author

Ah, I see, the cmake builds have placed the swift content under a swift directory rather than the top level.

@compnerd
Copy link
Member Author

Please test with following PR:
swiftlang/swift-corelibs-xctest#216

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 184c41e9f9805c0d45cadf5eb3a433c14f7dc105

@compnerd
Copy link
Member Author

CC: @aciidb0mb3r - any idea why s-p-m fails to find xctest?

@lanza
Copy link
Contributor

lanza commented Jun 25, 2018

@swift-ci please test Linux platform

1 similar comment
@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 184c41e9f9805c0d45cadf5eb3a433c14f7dc105

@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 184c41e9f9805c0d45cadf5eb3a433c14f7dc105

@compnerd
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 1eefac0fb0968b339c5f8b3772d224cd779f7bd4

@lanza
Copy link
Contributor

lanza commented Aug 22, 2018

Please test with following pull request:
swiftlang/swift-corelibs-xctest#220

@swift-ci Please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 1eefac0fb0968b339c5f8b3772d224cd779f7bd4

@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 1eefac0fb0968b339c5f8b3772d224cd779f7bd4

@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 1eefac0fb0968b339c5f8b3772d224cd779f7bd4

Now that XCTest has a CMake build system, use that for the non-Darwin
builds.  This sets the stage for building Foundation with CMake.
@compnerd
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 1eefac0fb0968b339c5f8b3772d224cd779f7bd4

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@gottesmm
Copy link
Contributor

gottesmm commented Sep 1, 2018

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.

@gottesmm
Copy link
Contributor

gottesmm commented Sep 1, 2018

To be clear, if I am correct that this is a dependency problem, we should eliminate that dependency problem first before this merges.

@gottesmm
Copy link
Contributor

gottesmm commented Sep 1, 2018

@swift-ci clean test linux platform

@swift-ci
Copy link
Contributor

swift-ci commented Sep 1, 2018

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@gottesmm
Copy link
Contributor

gottesmm commented Sep 5, 2018

@swift-ci test linux platform

1 similar comment
@gottesmm
Copy link
Contributor

gottesmm commented Sep 5, 2018

@swift-ci test linux platform

@gottesmm
Copy link
Contributor

gottesmm commented Sep 5, 2018

Log was eliminating. I am re-starting the incremental build to see if we can reproduce on the bots again.

@swift-ci
Copy link
Contributor

swift-ci commented Sep 5, 2018

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@compnerd
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@compnerd
Copy link
Member Author

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@compnerd
Copy link
Member Author

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 519595d

@compnerd
Copy link
Member Author

Please test with following PR:
swiftlang/swift-corelibs-xctest#224

@swift-ci please test

@compnerd compnerd merged commit f8d2745 into swiftlang:master Sep 21, 2018
@compnerd compnerd deleted the xctest-cmake branch September 21, 2018 17:01
@gottesmm
Copy link
Contributor

@compnerd congrats!

@jrose-apple
Copy link
Contributor

Looks like XCTest is no longer getting rebuilt when the standard library changes. What can we do about that?

@jrose-apple
Copy link
Contributor

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.

6 participants