Support debugging
akurtakov opened this issue · 7 comments
There is Ruby debug adapter protocol implementation now https://github.com/castwide/readapt and lsp4e supports one too so it should be possible to add support now.
Hey @akurtakov! Yes, I'm aware of Readapt and actually started implementing support a few weeks back. It didn't quite work then but I believe I was almost there: debugging Ruby code through Eclipse was functional, however, for some reason I had to start Readapt through the command line for things to work properly. This was back in August in the very early days of Readapt, so I decided to stash the changes locally and focus on other work in the meantime, but I'm hoping to pick this up again soon. 😉
I'm still having the same issues with the latest version of Readapt. I've pushed my work in progress to a branch (https://github.com/PyvesB/eclipse-solargraph/tree/debug) and sent a mail to the LSP4E list.
Okay, so I think this is simply due to the fact that Readapt does currently not support communication over stdin/stdout. I've opened the following issue: castwide/readapt#3
Progress:
It works, but the user experience is not great. Either the linked Readapt issue or Bug 552451 would have to be solved to make this more viable.
Okay, so I've implemented debugging support in the above commit! 🎉
I'm not planning on releasing this just yet: it currently requires to manually build and install Readapt from the master branch. There also are a few outstanding issues I'm discussing with the author. Once a new release of Readapt is published to rubygems.org, this plugin will automatically install it and wire things up into Eclipse with the code I've pushed.
There will probably be things to refine and improve at a later point. But for now, I would say initial debugging support is implemented, so I'll close this issue. 😉
Is there progres on release with debug support?
Is there progres on release with debug support?
All the code is ready on my side. There were a few issues with the Readapt implementation which I helped investigate and test in recent weeks (see castwide/readapt#3). The last remaining bit is now a new release of the Readapt gem, which is planned before the end of the month, but that's outside my control. 😉