Skip to content

[CMake] Don't remove sourcekit-inproc from _SWIFT_DEFAULT_COMPONENTS in SwiftComponents.cmake#88159

Merged
rintaro merged 1 commit intoswiftlang:mainfrom
Steffeeen:sourcekitdinproc-framework
Mar 30, 2026
Merged

[CMake] Don't remove sourcekit-inproc from _SWIFT_DEFAULT_COMPONENTS in SwiftComponents.cmake#88159
rintaro merged 1 commit intoswiftlang:mainfrom
Steffeeen:sourcekitdinproc-framework

Conversation

@Steffeeen
Copy link
Copy Markdown
Member

SourceKit always needs sourcekit-inproc. With this change we always get a lib/sourcekitdInProc.framework directory in the swift build directory (e.g., build/Ninja-RelWithDebInfoAssert/swift-macosx-arm64). This can then be used in SourceKit-LSP to run SourceKitD in-process on macOS for debugging. For more context see swiftlang/sourcekit-lsp#2580.

Comment thread cmake/modules/SwiftComponents.cmake Outdated
@rintaro
Copy link
Copy Markdown
Member

rintaro commented Mar 27, 2026

@swift-ci Please build toolchain macOS

@Steffeeen Steffeeen force-pushed the sourcekitdinproc-framework branch from 773fc1c to e284812 Compare March 27, 2026 16:45
SourceKit always needs sourcekit-inproc. With this change we always get
a `lib/sourcekitdInProc.framework` directory in the swift build
directory. This can then be used in SourceKit-LSP to run SourceKitD
in-process on macOS for debugging.
@Steffeeen Steffeeen force-pushed the sourcekitdinproc-framework branch from e284812 to dd51213 Compare March 27, 2026 16:45
@rintaro
Copy link
Copy Markdown
Member

rintaro commented Mar 27, 2026

@swift-ci Please smoke test

@rintaro
Copy link
Copy Markdown
Member

rintaro commented Mar 27, 2026

(toolchain build is running in https://ci.swift.org/job/swift-PR-toolchain-macos/2281/)

@rintaro rintaro merged commit 8619bf1 into swiftlang:main Mar 30, 2026
3 checks passed
@rintaro
Copy link
Copy Markdown
Member

rintaro commented Mar 30, 2026

It turned out that, toolchain builds are not affected by this because they are explicitly specifying the components. In practice, the only case affected are at-desk builds.

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.

2 participants