all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: "Kai Großjohann" <kai@emptydomain.de>, rms@gnu.org, emacs-devel@gnu.org
Subject: Re: Tramp and VC integration: "calling user"
Date: Fri, 01 Apr 2005 14:14:19 -0500	[thread overview]
Message-ID: <jwvd5tefh91.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <1112377387.17621.111.camel@localhost> (Andre Spiegel's message of "Fri, 01 Apr 2005 19:43:07 +0200")

>> (1) New file operation file-mine-p, returns true if the file is owned
>> by the "calling user".  For non-special files, the calling user is
>> the user who invoked Emacs.  For Tramp files, the calling user is
>> the user logged into the remote host.
>> 
>> (2) New file operation file-calling-user, returns the calling user, as
>> defined in (1).
>> 
>> (3) Augment the return value of file-remote-p to indicate the calling
>> user.  The return value could be augmented to also indicate the
>> remote host, if the file is remote.
>> 
>> #3 seems kludgy, so it shouldn't be that.  I prefer #1.  

> But #1 is in fact wrong.  It is irrelevant who the owner of the file is
> (the same argument as I made concerning file-writable-p).  What must be
> tested is whether the name of the locking user, as recorded in the RCS
> master file, is that of the calling user.  I still think #2 is the best
> way to achieve this.

Am I right in believing that this is a problem specific to 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?
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.

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.


        Stefan

  reply	other threads:[~2005-04-01 19:14 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-31 13:07 Tramp and VC integration: "calling user" Kai Großjohann
2005-03-31 13:34 ` Stefan Monnier
2005-03-31 14:40   ` Kai Großjohann
2005-03-31 15:50     ` Stefan Monnier
2005-03-31 17:13       ` Kai Großjohann
2005-03-31 16:54   ` Andre Spiegel
2005-04-01  4:10 ` Richard Stallman
2005-04-01 17:43   ` Andre Spiegel
2005-04-01 19:14     ` Stefan Monnier [this message]
2005-04-02 20:52       ` Andre Spiegel
2005-04-03 17:19         ` Michael Albinus
2005-04-03  5:19     ` Richard Stallman
2005-04-03 10:15 ` Kai Großjohann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwvd5tefh91.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=kai@emptydomain.de \
    --cc=rms@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.