Subject: bug#72117: Command doesn't execute correctly in eshell

From: Jim Porter <[email-masked]>
Date: Thu, 18 Jul 2024 08:38:53 -0700

On 7/18/2024 2:31 AM, Michael Albinus via Bug reports for GNU Emacs, the 
Swiss army knife of text editors wrote:
> Jim Porter <[email-masked]> writes:
> 
>> diff --git a/lisp/eshell/esh-proc.el b/lisp/eshell/esh-proc.el
>> index 0dcdf3bb76c..fbeb13362f3 100644
>> --- a/lisp/eshell/esh-proc.el
>> +++ b/lisp/eshell/esh-proc.el
>> +                   (cond
>> +                    ;; Delay signalling remote processes to prevent
>> +                    ;; "Forbidden reentrant call of Tramp".
>> +                    ((process-get proc 'remote-pid)
>> +                     (run-at-time 0 nil #'signal-process proc 'SIGPIPE))
> 
> Shouldn't there be a non-nil REMOTE argument?

Isn't a process object with the 'remote-pid' property set sufficient 
here? From my understanding of the code, REMOTE is redundant in that case.

(I'll probably need the REMOTE argument in some other Eshell code that 
we've been discussing recently, but not here I think...)