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

Docker remote debugging support #463

Closed
factyy opened this issue Apr 16, 2021 · 7 comments
Closed

Docker remote debugging support #463

factyy opened this issue Apr 16, 2021 · 7 comments

Comments

@factyy
Copy link
Contributor

factyy commented Apr 16, 2021

Hi everyone!

I want to help (if I have the time though) with dockerized debugging (both an app and a debug server residing inside a container). Any tips or hints?

Edit: this is a part of my lsp-mode wishlist suggestions: emacs-lsp/lsp-mode#515 (comment)

@yyoncho
Copy link
Member

yyoncho commented Apr 17, 2021

I will do it like this:

  1. Change dap-mode to be able to start remote debug adapter and communicate with it.
  2. Set dap-print-io to t and inspect that we are sending the correct local path to the debug adapter. We can put in the debug session function which translates the paths or we can eventually use that same function but from the lsp--client that is used by the current fine. I am not sure which approach will be better.

@adasium
Copy link

adasium commented Jul 15, 2021

There will also be a need to store currently used template somewhere, right? Since the user might define various mappings for different templates.

@yyoncho
Copy link
Member

yyoncho commented Jul 16, 2021

@milochadam yes.

@factyy
Copy link
Contributor Author

factyy commented Jan 28, 2022

@yyoncho I started to work on this feature at last. Decided to stick with option 2: using lsp--client features (lsp--path-to-uri with DAP mode changed to "uri" from "path" in the initialization request)

@factyy
Copy link
Contributor Author

factyy commented Jan 28, 2022

And quickly got a problem that not all debug providers support URIs...
Ok, we need new translation functions (the ones provided by lsp-mode work only with URIs)

@yyoncho
Copy link
Member

yyoncho commented Jan 28, 2022

@factyy we can transform to uri and back I guess.

@factyy
Copy link
Contributor Author

factyy commented Feb 14, 2022

Filed a PR #559, closing this issue

@factyy factyy closed this as completed Feb 14, 2022
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

No branches or pull requests

3 participants