Skip to content
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

[SR-12491] Create a branch of sourcekit-lsp setup with the latest released Swift toolchain #536

Closed
benlangmuir opened this issue Apr 2, 2020 · 4 comments
Labels
bug Something isn't working

Comments

@benlangmuir
Copy link
Member

Previous ID SR-12491
Radar rdar://problem/62201105
Original Reporter @benlangmuir
Type Bug
Additional Detail from JIRA
Votes 0
Component/s SourceKit-LSP
Labels Bug
Assignee None
Priority Medium

md5: 7be0189d3ec7cd413edbff041349f37e

Issue Description:

It would be useful to have a branch (e.g. master-with-swift-5.2) that contains the latest changes to sourcekit-lsp, but built to work with the currently released swift-5.2 toolchain. Currently we only have master (which works with master swift/swiftpm) and swift-5.2-branch (which works with swift-5.2, but does not receive new changes since it is already released).

Specifics

  • Create a branch master-with-swift-5.2 that uses master sourcekit-lsp, but swiftpm from 5.2.

  • Add PR testing for this branch that uses the latest released toolchain. This could be simpler than the existing PR testing and just use the downloadable toolchain instead of build swift from source.

  • Add a gated automerger from master to master-with-swift-5.2 if it tests successfully.

  • When we release swift-5.3, end support for 5.2 and move to master-with-5.3.

@akyrtzi
Copy link
Member

akyrtzi commented Apr 2, 2020

Do we need separate branches per release, why not create a `master-with-released-swift` branch?

This would be the branch that works with latest GM Xcode toolchain. Someone that wants to use latest development SKLSP with GM Xcode can use that one branch, and not have to figure out what branch they need to checkout and remember to change it when new GM comes out.

@benlangmuir
Copy link
Member Author

I'd be fine either way.

Pros for master-with-released:

  • Only one branch name to learn

Pros of master-with-swift-5.2:

  • No risk that you didn't notice what toolchain it's supposed to work with if there are e.g. betas or you're on release N-1 and haven't updated yet.

Another option would be for master-with-released to be master and we have master-with-master (or some better name) for the one we build with swift master. That would further simplify things for first timers building sourcekit-lsp, but makes it a little harder to understand and work with for people who are working on the rest of the swift toolchain.

@beccadax
Copy link

@swift-ci create

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@shahmishal shahmishal transferred this issue from apple/swift May 9, 2022
@ahoppen
Copy link
Collaborator

ahoppen commented Aug 22, 2022

Since the requirement for this never really came up in the last couple of years, I’m going to close this. AFAICT the current main SourceKit-LSP builds with the last resealed Swift toolchain.

@ahoppen ahoppen closed this as completed Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants