Skip to content

Conversation

rintaro
Copy link
Member

@rintaro rintaro commented Oct 7, 2025

Cherry-pick #2300 into release/6.2

  • Explanation: Previously in SourceKitD.getOrCreate(), setenv(3) was called with a key that might contain = character. In such cases, setenv failed an Error was thrown and failed. That logic was in a compatibility logic that was only used when sourcekit-lsp was used with older version of sourcekitd, which is not likely to happen. So cherry-pick the commit removing that logic. This change also obsolutes sourcekitd_plugin_initialize which was also used by only older sourcekitd.
  • Scope: SourceKit plugins
  • Risk: Low. We are sure the environment variable in question was not actually used.
  • Testing: Passed existing test cases
  • Issues: rdar://161758284
  • Reviewers: Alex Hoppen as the author of the original PR Drop support for building with Swift 6.1 #2300

Swift 6.2 has been released, so we no longer need to support building and testing SourceKit-LSP using Swift 6.1

(cherry picked from commit c7ca73a)
@rintaro rintaro requested a review from a team as a code owner October 7, 2025 04:47
@rintaro
Copy link
Member Author

rintaro commented Oct 7, 2025

@swift-ci Please test

Package.swift Outdated
@@ -1,4 +1,4 @@
// swift-tools-version: 6.1
// swift-tools-version: 6.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably leave 6.2 on 6.1

@rintaro
Copy link
Member Author

rintaro commented Oct 7, 2025

@swift-ci Please test

@rintaro
Copy link
Member Author

rintaro commented Oct 7, 2025

@swift-ci Please test Windows

1 similar comment
@rintaro
Copy link
Member Author

rintaro commented Oct 7, 2025

@swift-ci Please test Windows

@rintaro rintaro enabled auto-merge October 7, 2025 15:39
@rintaro rintaro merged commit bdd5f6e into swiftlang:release/6.2 Oct 7, 2025
3 checks passed
@rintaro rintaro deleted the 6.2-drop-6.1 branch October 7, 2025 18:23
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.

3 participants