From: Juliusz Chroboczek <Juliusz.Chroboczek@pps.jussieu.fr>
Cc: Andre Spiegel <spiegel@gnu.org>, emacs-devel@gnu.org
Subject: Re: Limitations of Emacs' vc when using modern backends
Date: Thu, 15 Dec 2005 18:55:17 +0100 [thread overview]
Message-ID: <7ilkym1bzu.fsf@lanthane.pps.jussieu.fr> (raw)
In-Reply-To: <87r78e1e4v.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Thu, 15 Dec 2005 12:33:42 -0500")
>> If multiple revisions match, the latest one is chosen.
> And this is a feature of vc-darcs rather than of darcs itself, right?
No, it's actually a feature of Darcs itself.
> The same thing proably happens for you: "Update" may stand for
> "Update version number to 1.3" at some point and to "Update version
> number to 1.6" at the other, so the "vc-darcs.el~Update" can become
> out of date as well.
Yes, exactly.
> The only problem I see with `canonical-revision' is that for some backends
> it may be very costly (for CVS it requires a round trip to the repository),
> and it may suffer from race conditions.
Good point about the cost (see below). On the other hand, there's no
race condition if you do things in the right order: first normalise
the version, then fetch the file from the backend using the canonical
identifier.
> Now `canonical-revision' is more general, but I'm wondering: where would it
> be used other than in `vc-version-other-window'?
In the current vc-darcs, it's also used by vc-diff. I haven't checked
exhaustively if there are other instances where it might be useful.
I suggest that for backends that need it vc-BACKEND-canonical-revision
should be compulsory, and we should optionally allow checkout and diff
to return the canonical identifier(s). (What a pity elisp doesn't
have multiple return values). I believe that the former should be
considered for 22, while the latter is something we might need more
time to experiment with.
And adding generic support for completion is definitely a good idea.
Bug again it might be premature to aim for having it in 22.
Juliusz
next prev parent reply other threads:[~2005-12-15 17:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-12-15 1:42 Limitations of Emacs' vc when using modern backends Juliusz Chroboczek
2005-12-15 3:37 ` Stefan Monnier
2005-12-15 14:59 ` Juliusz Chroboczek
2005-12-15 17:33 ` Stefan Monnier
2005-12-15 17:55 ` Juliusz Chroboczek [this message]
2005-12-15 20:24 ` Stefan Monnier
2005-12-15 14:37 ` Andre Spiegel
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=7ilkym1bzu.fsf@lanthane.pps.jussieu.fr \
--to=juliusz.chroboczek@pps.jussieu.fr \
--cc=emacs-devel@gnu.org \
--cc=spiegel@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 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).