Subject: Re: bug#30434: magit won’t work over TRAMP

From: Maxim Cournoyer <[email-masked]>
Date: Thu, 21 Jul 2022 00:04:06 -0400

Hi Maxime,

Maxime Devos <[email-masked]> writes:

[...]

>> Nowadays 'magit' has a separate magit-git-executable:
>>
>>    "The Git executable used by Magit on the local host.
>> On remote machines `magit-remote-git-executable' is used instead."
>>
>> and magit-remote-git-executable:
>>
>> (defcustom magit-remote-git-executable "git"
>>    "The Git executable used by Magit on remote machines.
>> On the local host `magit-git-executable' is used instead.
>> Consider customizing `tramp-remote-path' instead of this
>> option."
>>
>> so maybe this patch can now be reversed, such that emacs-magit
>> can be used without depending on the (possibly non-existent) 'git' in
>> $PATH?  Needs to be verified though.
>
> More concretely, try "guix shell emacs emacs-magit --pure -- emacs"
> followed by "M-x magit-status" in a Git checkout, it will fail due to
> not finding the 'git' executable.
>
> So my idea is to use the new magit changes to both make the remote
> TRAMP work and _also_ make local things work in a pure environment,
> undoing the regression that was caused by reverting the
> git->/gnu/store/.../bin/git substitution without creating new
> regressions.

Sounds good to me!  I'll be happy to review any patch implementing it.

Maxim