unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@IRO.UMontreal.CA>
To: Michael Albinus <michael.albinus@gmx.de>
Cc: Julian Scheid <julians37@googlemail.com>,
	"emacs-devel@gnu.org" <emacs-devel@gnu.org>
Subject: Re: process-file and Tramp's file cache
Date: Tue, 18 Aug 2009 12:23:14 -0400	[thread overview]
Message-ID: <jwvocqdumjc.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <nqljlhmatp.fsf@alcatel-lucent.de> (Michael Albinus's message of "Tue, 18 Aug 2009 16:54:42 +0200")

>> How 'bout changing Tramp's process-file to compare the command (and
>> args) against a list of "known pure commands"?  This would keep it all
>> inside Tramp, which I think I like better (tho I also agree that this
>> information is not fundamentally specific to Tramp, although currently
>> Tramp is the only package able to make use of it).

> That is also Julian's proposal. I dislike it, because there might be
> many packages Tramp has never heard about. And even for the packages
> contained in core Emacs, it will be an endless story: think about all the
> vc-BACKEND calls of list whatever.

OTOH, this list could be kept in a variable, so external packages could
easily add to it.

> The charme is also, that packages can precisely say, which file has been
> changed.

Clearly, each option has its advantages.  I'd mostly like to avoid
overengineering it, so for now I think a boolean is plenty (after all,
we currently live fairly happily with a worst-case behavior).

> Parsing `args' of `process-file' might also not a good idea,
> I believe.

It could prove problematic, indeed (I'm thinking mostly of having to
search for "status" in the argument of "cvs", having to skip the options
that might come before).

Let's go with your suggestion, but limited to a boolean for now (so the
name shouldn't end with "-list").


        Stefan




  reply	other threads:[~2009-08-18 16:23 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <87vdt2ndkt.fsf@tux.homenetwork>
     [not found] ` <200904071657.n37GvIau023055@godzilla.ics.uci.edu>
     [not found]   ` <87d4bnfkxn.fsf@tux.homenetwork>
     [not found]     ` <200904081624.n38GO6D6013154@godzilla.ics.uci.edu>
     [not found]       ` <jwvbpr7c9sd.fsf-monnier+emacsbugreports@gnu.org>
     [not found]         ` <nq4owy7x5y.fsf@alcatel-lucent.de>
     [not found]           ` <jwvy6u999m6.fsf-monnier+emacsbugreports@gnu.org>
     [not found]             ` <nqmyap7tpj.fsf@alcatel-lucent.de>
     [not found]               ` <jwvy6u97kqn.fsf-monnier+emacsbugreports@gnu.org>
     [not found]                 ` <87myalrb5w.fsf@gmx.de>
     [not found]                   ` <jwv1vrxzmoz.fsf-monnier+emacsbugreports@gnu.org>
     [not found]                     ` <87r5zxprx9.fsf@gmx.de>
     [not found]                       ` <jwv8wm5xzqq.fsf-monnier+emacsbugreports@gnu.org>
     [not found]                         ` <87prfgzys9.fsf@gmx.de>
     [not found]                           ` <jwvprfgwxmp.fsf-monnier+emacsbugreports@gnu.org>
     [not found]                             ` <nqtz4ruut9.fsf@alcatel-lucent.de>
     [not found]                               ` <jwveivvrww9.fsf-monnier+emacsbugreports@gnu.org>
2009-08-18 10:16                                 ` process-file and Tramp's file cache (was: bug#1741: 23.0.60; vc fail to register file while logged as /su:: with tramp) Michael Albinus
2009-08-18 14:32                                   ` process-file and Tramp's file cache Stefan Monnier
2009-08-18 14:54                                     ` Michael Albinus
2009-08-18 16:23                                       ` Stefan Monnier [this message]
2009-08-18 18:49                                         ` Michael Albinus
2009-08-18 19:19                                           ` Stefan Monnier
2009-08-19  7:44                                             ` Michael Albinus

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=jwvocqdumjc.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=julians37@googlemail.com \
    --cc=michael.albinus@gmx.de \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).