-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[build-script] When building packages install libIndexStore from clang #20132
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
[build-script] When building packages install libIndexStore from clang #20132
Conversation
This is needed for the LSP service.
@swift-ci build linux toolchain |
@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.
LGTM!
utils/build-presets.ini
Outdated
@@ -1062,7 +1062,7 @@ test-installable-package | |||
reconfigure | |||
|
|||
swift-install-components=compiler;clang-builtin-headers;stdlib;swift-syntax;sdk-overlay;license;sourcekit-xpc-service;swift-remote-mirror;swift-remote-mirror-headers | |||
llvm-install-components=llvm-cov;llvm-profdata | |||
llvm-install-components=llvm-cov;llvm-profdata;install-IndexStore |
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.
Shouldn't that be IndexStore
and not install-IndexStore
?
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.
Good catch!
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.
Oooops, my bad.
@akyrtzi Are you sure that install-IndexStore is right? It should be just IndexStore. |
@compnerd lol?! Who was first? Not sure. = p. |
@swift-ci build toolchain |
macOS Toolchain Install command |
Linux Toolchain (Ubuntu 16.04) Install command |
@akyrtzi Not sure, why GitHub did not update the status on the PR. |
Toolchains look good. |
This is needed for the LSP service.