-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[cmake] Move a bunch of test binaries from install component 'tools' -> 'testsuite-tools'. #40077
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
[cmake] Move a bunch of test binaries from install component 'tools' -> 'testsuite-tools'. #40077
Conversation
@swift-ci test |
@swift-ci build toolchain |
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.
Thanks! This definitely should help improve the CI space usage on Windows.
Linux Toolchain (Ubuntu 16.04) Install command |
macOS Toolchain Install command |
Some measurements - this reduced the build artifacts from 3.9G to 3.5G. |
@compnerd just a FYI, I am checking with some people to make sure my guesstimates on what we can remove here is correct. |
@gottesmm - sounds good; I would be more clear though - it isn't being removed, only being given additional control when building with the LLVM distribution mechanism. If you use a standalone Swift build, this will not have any impact. |
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.
LGTM for swift-ide-test
, swift-syntax-parser-test
and swift-syntax-test
.
…-> 'testsuite-tools'. All of these are tools that are only meant to be used when testing swift. Thus it doesn't make sense to include them in the catch all 'tools' install component that distributions use to build all the tools. I verified that all of the mac/linux presets in tree that have tools also has testsuite-tools so this should be NFC. On Windows, I needed to add testsuite-tools to build-windows.bat so should be NFC there as well.
df5f9f5
to
06cb5c0
Compare
I removed swift-ide-test since I think some people depend on it. I think I am going to include it in a different category though. |
@swift-ci smoke test |
All of these are tools that are only meant to be used when testing swift. Thus
it doesn't make sense to include them in the catch all 'tools' install component
that distributions use to build all the tools.
I verified that all of the mac/linux presets in tree that have tools also has
testsuite-tools so this should be NFC. On Windows, I needed to add
testsuite-tools to build-windows.bat so should be NFC there as well.