Fix pet.el to work on tramp

wyuenho / emacs-pet

Tracks down the correct Python tooling executables from your virtualenvs so you can glue the binaries to Emacs and delete code in init.el

GNU General Public License v3.0

105 stars 13 forks source link

Fix pet.el to work on tramp #20

Closed egelja closed 1 year ago

egelja commented 1 year ago

Sets the remote parameter of executable-find to t (only works in Emacs 27+)
Uses process-file instead of call-process
Checks if there is a remote connection when finding the bin directory
Also fixes arguments to dasel

Closes #19

egelja commented 1 year ago

@wyuenho Could you please take a look at this?

wyuenho commented 1 year ago

Hi sorry for the delay. I don't use tramp, and I honestly have no idea how to set up an environment to test this. Is this about using dev containers like the cool kids do these day? If so, can you please provide me with some detailed instructions so I can play around with this PR? Thanks.

egelja commented 1 year ago

I don't use it for dev containers. Just for editing files on a remote server. To do so, just use C-x C-f /ssh:<SERVER>:<PATH>. Then you can check if pet is working with pet-verify-setup.

egelja commented 1 year ago

Just updated with the current main branch.