Re: [Emacs-diffs] /srv/bzr/emacs/trunk r109810: * eshell/esh-ext.el (eshell-external-command): Do not examine

[Michael Albinus]

Stefan Monnier <address@hidden> writes:

>> +  (let ((interp (eshell-find-interpreter
>> +             command
>> +             ;; Do not examine remote shell scripts.
>> +             (or (and (stringp command) (file-remote-p command))
>> +                 (file-remote-p default-directory)))))
>
> I think that most uses of file-remote-p are wrong (in the sense that,
> while they may return an info that's correlated with the one we need,
> it's not exactly what we need) and this one is no exception.
> So I think in the above case it'd be better to use something along the
> lines of `ignore-errors'.

We have different interpretation of `file-remote-p', yes.

But in this given case, I do not understand what you mean with "along
the lines of `ignore-errors'". If I understand correctly the reported
error, we must suppress eshell's feature to look for the shell
interpreter in files located on a remote host. How shall I implement
this with the help of `ignore-error'?

>         Stefan

Best regards, Michael.