From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Andre Spiegel Newsgroups: gmane.emacs.devel Subject: Re: Tramp and VC integration: "calling user" Date: Sat, 02 Apr 2005 22:52:03 +0200 Message-ID: <1112475123.17621.255.camel@localhost> References: <87is386k0h.fsf@emptydomain.de> <1112377387.17621.111.camel@localhost> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit X-Trace: sea.gmane.org 1112475953 29805 80.91.229.2 (2 Apr 2005 21:05:53 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Sat, 2 Apr 2005 21:05:53 +0000 (UTC) Cc: Kai =?ISO-8859-1?Q?Gro=DFjohann?= , michael.albinus@gmx.de, rms@gnu.org, emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Apr 02 23:05:49 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1DHpoG-0002DL-Gg for ged-emacs-devel@m.gmane.org; Sat, 02 Apr 2005 23:05:37 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DHppn-0005Lp-UE for ged-emacs-devel@m.gmane.org; Sat, 02 Apr 2005 16:07:12 -0500 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DHppM-00056P-T6 for emacs-devel@gnu.org; Sat, 02 Apr 2005 16:06:45 -0500 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DHppG-00052T-1x for emacs-devel@gnu.org; Sat, 02 Apr 2005 16:06:41 -0500 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DHppF-0004z5-Rn for emacs-devel@gnu.org; Sat, 02 Apr 2005 16:06:37 -0500 Original-Received: from [193.113.160.39] (helo=mail.o2.co.uk) by monty-python.gnu.org with esmtp (Exim 4.34) id 1DHpbR-0004AG-Bd; Sat, 02 Apr 2005 15:52:21 -0500 Original-Received: from [217.231.190.112] (217.231.190.112) by mail.o2.co.uk (7.0.042) id 4240277700171F0E; Sat, 2 Apr 2005 21:52:11 +0100 Original-To: Stefan Monnier In-Reply-To: X-Mailer: Evolution 2.0.4 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:35511 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:35511 On Fri, 2005-04-01 at 14:14 -0500, Stefan Monnier wrote: > Am I right in believing that this is a problem specific to vc-rcs.el? I know of no other occurences, and I'd be happy to localize changes into vc-rcs.el as much as possible. Michael Albinus has stated, however (on tramp-devel), that "such a function is on my wishlist for a while, not only because of vc." He didn't give any more details, so I'll put him in copy and ask him to let us know why he would like something like "file-calling-user", except for the use in vc-rcs.el. > Couldn't vc-rcs.el use RCS directly instead of reproducing RCS's > behavior? > I.e. just try a dry-run of `co', and see if it fails? I did things like that when I first made patches to VC, but we decided against that at the time; it was just too slow, although it is perhaps clean conceptually. The "dry-run" of co seems very much like a kludge to me, though. > Or otherwise run (shell-command "id") since shell-command is a fileop that > can be caught by file-name-handlers and run on the remote host. That sounds like a very interesting alternative, maybe better than file-calling-user. > I just feel that adding a global command is difficult to justify just for > the odd case where all the following conditions are met: > - you're using vc-rcs.el. > - you're using it over Tramp. > - you've played with the permissions such that they don't give us directly > the proper information about whether a file is locked. Well number one is not odd at all, RCS is still a widely used version control systems at many sites. Number three is also not very unlikely -- as I said, I've seen things like that happen at almost every site where RCS is used. It is very common that clueless users (or lazy ones) simply change the permissions of a file to make it "work", without realizing that they are circumventing RCS this way. (I got a bug report from someone who managed to get himself into trouble this way just this present week.) VC has the potential to handle these cases much more gracefully than RCS does on its own. Number two in your list is what's not working currently, and we're only trying to achieve it for reasons of consistency. As I said, I'd be happy to make the change as local to vc-rcs.el as possible. I can certainly envision Tramp-specific code in VC, because it's just another standard part of Emacs. But maybe a more general solution is in fact called for.