[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: Tue, 16 Jul 2013 10:44:57 +0200
User-agent: 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'.

>         Stefan

Best regards, Michael.

reply via email to

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