-
Notifications
You must be signed in to change notification settings - Fork 10.6k
Unskip building docc for cross-compiling on Windows #84912
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
This should be working now, let's try it...
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci test |
@swift-ci please build toolchain arm64 platform |
@swift-ci smoke test |
This doesn't work because we have only built SwiftPM itself for aarch64 (only the compiler has a x86_64 variant for the host, when cross compiling). DocC needs to adopt CMake. |
@swift-ci test |
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci please build toolchain arm64 platform |
@swift-ci smoke test |
@swift-ci test |
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci please build toolchain arm64 platform |
@swift-ci test |
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci please build toolchain arm64 platform |
@swift-ci smoke test |
@swift-ci test |
@swift-ci smoke test |
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci please build toolchain arm64 platform |
Now we keep the old behavior for running tests, while using the pinned toolchain for builds
@swift-ci test |
@swift-ci smoke test |
@swift-ci Please Build Toolchain Windows Platform |
@swift-ci please build toolchain arm64 platform |
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.
No, this is incorrect. We are using the just built runtime. We need to do that because this is used for testing Foundation and other bits that are part of the SDK and need to be tested against the just built runtime.
This should be working now, let's try it...