[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Emacs-diffs] trunk r113420: * net/tramp-sh.el (tramp-sh-handle-vc-r

From: Michael Albinus
Subject: Re: [Emacs-diffs] trunk r113420: * net/tramp-sh.el (tramp-sh-handle-vc-registered): Use `ignore-error'.
Date: Mon, 10 Jul 2017 17:53:38 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux)

Stefan Monnier <address@hidden> writes:

>>>> +    (ignore-errors
>>>> +      (tramp-run-real-handler 'vc-registered (list file))))))))
>>> Is it normal/common for (tramp-run-real-handler 'vc-registered (list
>>> file)) to signal an error in some corner cases?
>> No. I have added this as self-defense of Tramp.
> So an error here indicates an error in some code somewhere.  In this
> case, with-demoted-errors is preferable.

It took almost 4 years, but now I've fixed this.

>> The error was triggered, when I have opened a file on a NAS via Tramp.
>> I have mounted the emacs source directory there, and I have tried to
>> open a file in that directory.
>> Due to the existence of .../emacs/.bzr, there was a call
>>   (process-file "bzr" nil t nil "status" "--no-classify" "README")
>> bzr is not installed on that machine, so this call failed.
> Sounds like a bug in vc-bzr-register.  Could you either try and fix it
> there (catching the error right there), or file a bug report for it?

This bug doesn't happen anymore. I don't remember whether I've filed a
bug report, but it is fixed now.

>         Stefan

Best regards, Michael (proud, 'cos I've shortened my todo list).

reply via email to

[Prev in Thread] Current Thread [Next in Thread]