[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).
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Emacs-diffs] trunk r113420: * net/tramp-sh.el (tramp-sh-handle-vc-registered): Use `ignore-error'.,
Michael Albinus <=