Skip to content

[lldb] Look for lldb-dap instead of lldb-vscode in the toolchain #626

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

Closed
wants to merge 1 commit into from

Conversation

JDevlieghere
Copy link

The LLDB Debug Adaptor Protocol implementation was renamed from lldb-vscode to lldb-dap [1]. This commit updates the VSCode plugin to look for lldb-dap instead of lldb-vscode in the toolchain.

[1] https://discourse.llvm.org/t/rfc-rename-lldb-vscode-to-lldb-dap/74075

The LLDB Debug Adaptor Protocol implementation was renamed from
lldb-vscode to lldb-dap [1]. This commit updates the VSCode plugin to
look for lldb-dap instead of lldb-vscode in the toolchain.

[1] https://discourse.llvm.org/t/rfc-rename-lldb-vscode-to-lldb-dap/74075
@JDevlieghere
Copy link
Author

  • I haven't cherrypicked the upstream commit to any of the Swift branches yet.
  • I'd like to avoid having to add fallback logic to find both lldb-dap and lldb-vscode. It seems like the latter is only packaged with the Windows toolchain for now. If I cherrypick the rename to the 5.10 branch, does it sound reasonable to only look for lldb-dap? (@adam-fowler, @compnerd)

@compnerd
Copy link
Member

@JDevlieghere we are using the old name with the 5.9 toolchain already, we will need the fallback I fear.

@adam-fowler
Copy link
Contributor

I'd like to avoid having to add fallback logic to find both lldb-dap and lldb-vscode

No it's not ideal, but as @compnerd says it is already being used in Windows.

I don't know how many people are using it though. They could fall back to CodeLLDB for 5.9 maybe. The integration with lldb-vscode is only in a prerelease of the extension. Although changing it would mess with my release schedule slightly. I was hoping to do a full release with it being available. Managing multiple release branches is a pain. It would seem odd doing a full release of the extension with it using an executable not available in a swift release.

I would prefer if we had the fallback. It's not like the extension doesn't have tests for different swift versions elsewhere.

@adam-fowler
Copy link
Contributor

I'm happy to do the additional fallback work

@adam-fowler
Copy link
Contributor

@compnerd we could do this based on swift version. If we are using 5.9 look for lldb-vscode, otherwise look for lldb-dap. That would mean we would need an updated 5.10 toolchain before this was included in a release though. What do you think?

@compnerd
Copy link
Member

Yeah, that seems reasonable. The 5.10 toolchain is waiting on @shahmishal as we can no longer generate the toolchain Azure due to the dependency on the swift toolchain due to macros.

@shahmishal
Copy link
Member

You can track the work here for Windows nightly toolchain - swiftlang/swift#69484

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.

4 participants