unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Michael Albinus <michael.albinus@gmx.de>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: Alan Mackenzie <acm@muc.de>, Eli Zaretskii <eliz@gnu.org>,
	emacs-devel@gnu.org
Subject: Re: Optional argument for `file-local-copy'
Date: Sun, 30 Nov 2014 18:24:09 +0100	[thread overview]
Message-ID: <87k32cbo06.fsf@gmx.de> (raw)
In-Reply-To: <jwvy4qs7pad.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Sun, 30 Nov 2014 09:11:49 -0500")

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> Thanks for getting us back on track.  Yes, I agree with adding such
> a functionality.  But I have the impression that it's not enough: if the
> caller doesn't know whether the file is indeed being cached or not, then
> it doesn't know whether to delete it or not once it's done using it.

The caller does not need to know whether a file is cached. File name
handlers implementation shall keep *always* the information about a
local copy of a remote file (the local name, the result of
file-attributes of the remote file).

When file-local-copy is called with a non-nil REUSE-IF-ALREADY-EXISTS,
it checks whether there is a cached information about that remote file,
whether the local copy still exists, and whether the cached
file-attributes information is equal to the actual value. Then, and only
then, it reuses that local copy.

By this, any caller of file-local-copy could either ignore this new
argument, and delete the local copy after it was used, or the caller
does not delete the local copy because there is the assumption that the
local copy might be used, again. Candidates are retrieved web pages, or
remote files retrieved by a vc backend, like vc-bzr-admin-* files in
vc-bzr.el.

A problem might be to remove local copies when they are not used
anymore. This could be done when Emacs is stopped, and/or when Emacs is
started. But this shall be applied in the background, without
intervention of the user.

>         Stefan

Best regards, Michael.



  reply	other threads:[~2014-11-30 17:24 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-29 11:29 Optional argument for `file-local-copy' Michael Albinus
2014-11-29 13:03 ` Eli Zaretskii
2014-11-29 14:08   ` Alan Mackenzie
2014-11-29 14:39     ` Eli Zaretskii
2014-11-29 15:33       ` Alan Mackenzie
2014-11-29 16:24         ` Thien-Thi Nguyen
2014-11-30  5:22         ` Stefan Monnier
2014-11-30 11:04           ` Michael Albinus
2014-11-30 14:11             ` Stefan Monnier
2014-11-30 17:24               ` Michael Albinus [this message]
2014-12-02 14:18                 ` Stefan Monnier
2014-12-02 14:53                   ` Michael Albinus
2014-12-02 20:55                     ` Stefan Monnier
2014-12-02 21:24                       ` Michael Albinus
2014-12-02 22:30                         ` Stefan Monnier
2014-12-09 13:32                           ` Michael Albinus
2014-12-10 14:15                             ` Stefan Monnier
2014-12-11  9:20                               ` Michael Albinus
2014-12-11 15:22                                 ` Stefan Monnier
2014-12-14 10:22                                   ` Michael Albinus
2014-12-14 13:46                                     ` Stefan Monnier
2014-12-14 14:24                                       ` Michael Albinus
2014-12-14 22:37                                         ` Stefan Monnier
2014-12-15  7:41                                           ` Michael Albinus
2014-11-30 10:54         ` 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=87k32cbo06.fsf@gmx.de \
    --to=michael.albinus@gmx.de \
    --cc=acm@muc.de \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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).