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

add support for TRAMP #47

Open
bricewge opened this issue Jun 1, 2019 · 5 comments · May be fixed by #68
Open

add support for TRAMP #47

bricewge opened this issue Jun 1, 2019 · 5 comments · May be fixed by #68

Comments

@bricewge
Copy link
Contributor

bricewge commented Jun 1, 2019

Having support for TRAMP would allow the use of emacs-direnv on remote machines.

@wbolster
Copy link
Owner

i don't really see how that could work. this package patches the local env inside emacs, and inherently acts locally

@bricewge
Copy link
Contributor Author

bricewge commented Jul 6, 2019

I think it need to patch the remote tramp environment instead of the local when in a tramp directory. Some example of changes are call-process should be replaced by process-file and process-environment by tramp-remote-process-environment.

Some days ago I ran into a package that added tramp support with a couple of line patch, unfortunately I can't manage to find it today. The only lead I have ATM is that page page of the TRAMP manual on how to integrate it with other packages.

EDIT: In emacs 27.1 executable-find can take an optional remote argument, see here.

@eush77
Copy link
Contributor

eush77 commented Apr 20, 2020

I have just integrated Direnv with Tramp in my own configuration by prefixing commands executed in Tramp with direnv exec <default-directory>. I advised tramp-sh-handle-start-file-process instead of dealing with tramp-remote-process-environment, which goes quite contrary to how emacs-direnv operates and is less efficient, but it works.

@siddharthverma314 siddharthverma314 linked a pull request Aug 17, 2020 that will close this issue
@tshu-w
Copy link

tshu-w commented Jan 15, 2021

@wbolster any update? There seems to be a pull request.

@mrunhap
Copy link

mrunhap commented Apr 23, 2024

Any update? Two packages I know both don't support tramp, envrc and emacs-direnv.

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

Successfully merging a pull request may close this issue.

5 participants