manateelazycat / lsp-bridge

A blazingly fast LSP client for Emacs
GNU General Public License v3.0
1.42k stars 205 forks source link

lsp-bridge-find-def not working with remote #895

Closed leath-dub closed 5 months ago

leath-dub commented 6 months ago

when I run lsp-bridge-find-def in remote file it tries to go to a file in my local file system: not the remote. For example if I run it on a class A it trys to got to /usr/include/xxxx.h on my local file system not the remote file system. I am accessing the remote file using lsp-open-remote-file.

manateelazycat commented 6 months ago

Can you add some log in function lsp-bridge-define--jump ?

I guess something wrong to make code not execute https://github.com/manateelazycat/lsp-bridge/blob/22d0623378e77c8d20f1f5f8598ccf62bcf63e51/lsp-bridge.el#L1955

manateelazycat commented 6 months ago

@norris-young Can you help take a look at this issue? thank you for your time ;)

leath-dub commented 6 months ago

Can you add some log in function lsp-bridge-define--jump ?

I guess something wrong to make code not execute

https://github.com/manateelazycat/lsp-bridge/blob/22d0623378e77c8d20f1f5f8598ccf62bcf63e51/lsp-bridge.el#L1955

Sorry not using emacs anymore.

norris-young commented 6 months ago

@norris-young Can you help take a look at this issue? thank you for your time ;)

@manateelazycat @leath-dub Sorry for late reply. lsp-bridge-enable-with-tramp need to be set to nil which is default t if you want to use lsp-bridge-open-remote-file instead of opening with tramp. Maybe we need to add this limitation info to the README.

And this issue was mentioned before here.

manateelazycat commented 5 months ago

lsp-bridge-open-remote-file

Done

https://github.com/manateelazycat/lsp-bridge/commit/ee43c6caf78bd8ec851859e0dd690aa646231591