-
Notifications
You must be signed in to change notification settings - Fork 38
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
Install the SourceKit plugin into Windows toolchains #364
base: main
Are you sure you want to change the base?
Conversation
platforms/Windows/ide/ide.wxs
Outdated
@@ -37,6 +37,14 @@ | |||
<File Source="$(TOOLCHAIN_ROOT)\usr\lib\sourcekitdInProc.lib" /> | |||
</Component> | |||
|
|||
<Component Directory="_usr_lib"> |
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.
This seems like the wrong place ... DLLs are placed into the bin
directory, lib
should have static libraries and import libraries.
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.
These libraries will be opened by SourceKit using dlopen
and are not directly linked to SourceKit. Hence, they don’t ned to go into bin
for Windows to pick them up and I thus thought that it makes more sense to put them in lib
. Happy to change it, if you prefer to have them in bin
.
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.
I think that they should be in bin
. Unless they are completely standalone (including any system library dependencies). The current directory is scanned for any dependencies, and having them elsewhere makes that difficult.
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.
These libraries are fully standalone and don’t depend on any of the other libraries in bin
.
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.
But that also includes the C/C++ runtimes. They can also be redistributed in the toolchain (and if we run into the cases where we need to, we would fall back to that). In the mean time, note that other "plugin" libraries are loaded from bin
on Windows.
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.
Ah, I see. Moved it to bin
2ecee64
to
99384ee
Compare
Install the SourceKit plugin, added by swiftlang/sourcekit-lsp#1906 into the Windows toolchains. rdar://142909870
99384ee
to
7ff716c
Compare
Please do a cross-repository test with |
Install the SourceKit plugin, added by swiftlang/sourcekit-lsp#1906 into the Windows toolchains.
rdar://142909870