[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
Re: [Emacs-diffs] trunk r113420: * net/tramp-sh.el (tramp-sh-handle-vc-registered): Use `ignore-error'.
Tue, 16 Jul 2013 10:44:57 +0200
Gnus/5.13 (Gnus v5.13) Emacs/24.3.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.
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.
The real cause of the problem is a design deficiency, I believe. vc-bzr
should check, whether bzr is installed. Unfortunately, we have only
`executable-find', which works on the local machine. We would need a
counterpart of this function for remote machines, like the pairs
`call-process'/`process-file' and `start-process'/`start-file-process'.
Best regards, Michael.