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
next prev parent 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.