search

PyvesB / eclipse-solargraph

:gem: Ruby development tools for Eclipse.
https://marketplace.eclipse.org/content/ruby-solargraph
Eclipse Public License 2.0
30 stars 6 forks source link

Support debugging #3

Closed akurtakov closed 4 years ago

akurtakov commented 4 years ago

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.

PyvesB commented 4 years ago

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. 😉

PyvesB commented 4 years ago

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.

PyvesB commented 4 years ago

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: https://github.com/castwide/readapt/issues/3

PyvesB commented 4 years ago

Progress: readapt

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.

PyvesB commented 4 years ago

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. 😉

akurtakov commented 4 years ago

Is there progres on release with debug support?

PyvesB commented 4 years ago

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 https://github.com/castwide/readapt/issues/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. 😉