From: Johan Herland <johan@herland.net>
To: git@vger.kernel.org
Cc: Sverre Rabbelier <srabbelier@gmail.com>,
Jeff King <peff@peff.net>,
Nguyen Thai Ngoc Duy <pclouds@gmail.com>,
Nicolas Pitre <nico@fluxnic.net>
Subject: [1.8.0] Provide proper remote ref namespaces
Date: Wed, 02 Feb 2011 03:21:59 +0100 [thread overview]
Message-ID: <201102020322.00171.johan@herland.net> (raw)
In-Reply-To: <AANLkTimtU56BAnWU-2pY1npdkPdKEBq_CMCGwXUK+E=H@mail.gmail.com>
On Wednesday 02 February 2011, Sverre Rabbelier wrote:
> On Tue, Feb 1, 2011 at 19:14, Jeff King <peff@peff.net> wrote:
> > i.e., make refs/remotes/* an actual mirror of selected parts of the
> > remote's refs/ hierarchy. And then figure out sane rules for merging
> > those namespaces into the ref lookup procedure.
>
> Jeff, Nguy, are either of you interested in writing up a new/modifying
> this proposal to be about namespacing everything?
Here's my go at phrasing this in a proposal format. Feel free to revise and
resend:
Proposal:
Currently, git stores remote refs in the local repo by default as follows:
Remote repo -> Local repo
---------------------------------------------------------
HEAD refs/remotes/$remote/HEAD (implicit)
refs/heads/* refs/remotes/$remote/*
refs/tags/* refs/tags/* (implicit, autofollow)
refs/replace/* (TBD)
refs/notes/* (TBD)
Several users report that they are confused by the difference in how heads
and tags are mapped, and by the implicit mappings that are not mentioned in
the configured refspecs. Also, as users want to share ever more different
types of refs (replace refs and notes refs have been discussed recently),
the existing ref mappings (aka. refspecs) do not suggest a natural/intuitive
mapping for the new ref types.
Instead, we should change the default ref mappings into the following:
Remote repo -> Local repo
--------------------------------------------------
HEAD refs/remotes/$remote/HEAD
refs/heads/* refs/remotes/$remote/heads/*
refs/tags/* refs/remotes/$remote/tags/*
refs/replace/* refs/remotes/$remote/replace/*
refs/notes/* refs/remotes/$remote/notes/*
In short, we make refs/remotes/$remote/* an actual mirror of selected parts
of the remote's refs/* hierarchy. This provides consistent namespaces for
remote refs that naturally allows adding new ref types in the future.
This change obviously affects our ref-handling code:
- Remote tags are now stored separate from local tags. When looking up a
shorthand tag name (e.g. v1.7.4), we should consult local tags
(refs/tags/v1.7.4) before remote tags (refs/remotes/*/tags/v1.7.4 [1]). See
[2] for more details.
- Remote heads have moved into refs/remotes/$remote/heads/*, hence
invalidating shorthand remote head names, like "origin/master". We should
change the lookup code, so that a shorthand ref of the form "$remote/$head"
where "$remote" happens to match a configured remote is eventually expanded
into lookup for "refs/remotes/$remote/heads/$head" [3].
- We might want to generalize the handling of "$remote/$head" into allowing
shorthands like "$remote/$tag", "$remote/$replace" and "$remote/$note" as
well (provided, of course, that they match unambiguously).
- All fetch refspecs should be given explicitly.
Sub-proposal: While we are changing the default refspecs, we should also
consider whether we want to keep the auto-following behavior that Git
currently does for tags (don't fetch tags that refer to objects not
otherwise fetched by another refspec). If we simply make an explicit
"+refs/tags/*:refs/remotes/$remote/tags/*" refspec, we will lose the auto-
following behavior. If we do want to keep the auto-following behavior, we
could for example add a "~" prefix to the refspec to trigger auto-following
behavior (i.e. this refspec only applies to refs that happen to point at
objects fetched by way of a different refspec). See
http://thread.gmane.org/gmane.comp.version-control.git/160503/focus=160795
for more details.
Risks:
Existing scripts/programs may make assumptions about the layout of remote
refs without consulting the configured refspecs. However, such
scripts/programs may also break today when non-default refspecs are used.
When remotes have conflicting tags (same tag name points to different
objects), and the tag name does not exist locally (in refs/tags/*), looking
up the shorthand tag name will result in an "ambiguous ref" error (instead
of silently adopting whichever tag was fetched first). Although many
consider this an improvement on the current behavior, there may be scenarios
where this causes problems in external scripts/programs.
Existing scripts/programs that assume and depend on the current implicit
refspecs (or the tag auto-following behavior), might encounter problems when
we drop these in favor of explicit refspecs.
Migration plan:
The main part of this proposal is simply changing the default refspecs. As
such, the proposal can be simulated in current Git versions by setting up
custom refspecs according to the above table of ref mappings.
In v1.8.0, we should default to the new default refspecs when creating new
remotes. However, existing remotes (created pre-v1.8.0) must continue to
work as before, so we cannot simply remove the implicit refspecs (or tag
auto-following). Instead we need to make sure that the implicit refspecs is
NOT applied to the new-style remotes. Identifying new-style vs. old-style
remotes can be done by looking at the refspec itself (old-style:
"refs/remotes/$remote/*", new-style: "refs/remotes/$remote/heads/*"), or
(worst case) by introducing a config variable specifying the desired
behavior (defaulting to old-style).
When adding the new rules for looking up shorthand refs (described above),
we should carefully verify that these won't cause regressions when applied
to old-style refspecs in existing repos.
In a later major version we can consider removing the (by then ancient)
implicit refspecs, and any other outdated compatibility measures code in our
ref lookup code.
Have fun! :)
...Johan
[1]: The "refs/remotes/*/tags/v1.7.4" is not hardcoded, but rather the
(default) result of mapping "refs/tags/v1.7.4" through each remote's
refspecs as defined in the config.
[2]: When looking up a shorthand tag name (e.g. v1.7.4): If a local tag
(refs/tags/v1.7.4) is found, then we have an unambiguous match. If no local
tag is found, we look up the tag name in all configured remotes (using the
method described in [1]). If the tag name exists in one or more remotes, and
those remotes all agree on its ultimate object name (after applying e.g.
^{commit} or whatever is appropriate in the context of the lookup), then we
also have an unambiguous match. However, if the tag name exists in multiple
remotes, and they do NOT all agree on its ultimate object name, then the
shorthand tag name is ambiguous and the lookup fails. The user can always
resolve this ambiguity by creating a local tag (refs/tags/v1.7.4) pointing
to the desired object.
[3]: As in [1], the "refs/remotes/$remote/heads/$head" is not hardcoded, but
rather the result of mapping "refs/heads/$head" through the refspecs for
$remote as defined in the config.
--
Johan Herland, <johan@herland.net>
www.herland.net
next prev parent reply other threads:[~2011-02-02 2:22 UTC|newest]
Thread overview: 105+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-01 10:44 [1.8.0] Remote tag namespace Nguyen Thai Ngoc Duy
2011-02-01 14:10 ` Leo Razoumov
2011-02-01 15:07 ` Marc Branchaud
2011-02-01 15:35 ` Nguyen Thai Ngoc Duy
2011-02-02 19:38 ` Marc Branchaud
2011-02-01 18:14 ` Jeff King
2011-02-01 23:14 ` Sverre Rabbelier
2011-02-02 2:21 ` Johan Herland [this message]
2011-02-02 13:27 ` [1.8.0] Provide proper remote ref namespaces Santi Béjar
2011-02-02 15:51 ` Johan Herland
2011-02-02 16:19 ` Santi Béjar
2011-02-03 5:33 ` Nguyen Thai Ngoc Duy
2011-02-03 8:46 ` Johan Herland
2011-02-03 11:35 ` Nguyen Thai Ngoc Duy
2011-02-03 13:10 ` Johan Herland
2011-02-03 14:10 ` Santi Béjar
2011-02-03 15:48 ` Nguyen Thai Ngoc Duy
2011-02-04 22:39 ` Junio C Hamano
2011-02-05 1:18 ` Johan Herland
2011-02-05 18:00 ` Kevin P. Fleming
2011-02-05 21:40 ` Dmitry Potapov
2011-02-06 0:04 ` Johan Herland
2011-02-06 12:03 ` Dmitry Potapov
2011-02-06 14:09 ` Nicolas Pitre
2011-02-06 15:17 ` Dmitry Potapov
2011-02-06 16:11 ` Johan Herland
2011-02-06 17:28 ` Dmitry Potapov
2011-02-06 22:12 ` Johan Herland
2011-02-07 0:07 ` Dmitry Potapov
2011-02-07 19:05 ` Bernhard R. Link
2011-02-08 1:20 ` Dmitry Potapov
[not found] ` <201102070429.05033.johan@herland.net>
2011-02-07 3:47 ` Nicolas Pitre
2011-02-08 1:06 ` Dmitry Potapov
2011-02-08 8:15 ` Johan Herland
2011-02-08 19:11 ` Enrico Weigelt
2011-02-08 23:13 ` Junio C Hamano
2011-02-09 1:24 ` Johan Herland
2011-02-06 20:28 ` Matthieu Moy
2011-02-06 23:22 ` Johan Herland
2011-02-06 23:51 ` Matthieu Moy
2011-02-07 3:51 ` Johan Herland
2011-02-07 5:11 ` Jeff King
2011-02-07 8:58 ` Johan Herland
2011-02-07 9:01 ` Sverre Rabbelier
2011-02-07 10:06 ` Johan Herland
2011-02-07 10:22 ` Nguyen Thai Ngoc Duy
2011-02-07 20:19 ` Jeff King
2011-02-08 0:59 ` Johan Herland
2011-02-11 15:13 ` Jakub Narebski
2011-02-13 23:36 ` Johan Herland
2011-02-14 7:35 ` Junio C Hamano
2011-02-14 9:18 ` Johan Herland
2011-02-14 9:59 ` Jakub Narebski
2011-02-14 17:30 ` Junio C Hamano
2011-02-14 18:06 ` Re* " Junio C Hamano
2011-02-14 18:53 ` Jay Soffian
2011-02-14 19:44 ` Sverre Rabbelier
2011-02-14 19:50 ` Jay Soffian
2011-02-14 21:21 ` [1.8.0 RFC] push: start warning upcoming default change for push.default Jonathan Nieder
2011-02-14 21:41 ` Jay Soffian
2011-02-14 21:55 ` Jonathan Nieder
2011-02-14 21:57 ` Re* [1.8.0] Provide proper remote ref namespaces Matthieu Moy
2011-02-14 22:35 ` Jeff King
2011-02-14 22:38 ` Sverre Rabbelier
2011-02-14 23:36 ` [1.8.0 RFC] push: start warning upcoming default change for push.default Jonathan Nieder
2011-02-14 23:45 ` Re* [1.8.0] Provide proper remote ref namespaces Junio C Hamano
2011-02-14 23:05 ` Junio C Hamano
2011-02-15 15:06 ` Johan Herland
2011-02-15 18:06 ` Junio C Hamano
2011-02-16 0:54 ` [PATCH] push.default: Rename 'tracking' to 'upstream' Johan Herland
2011-02-16 6:35 ` Junio C Hamano
2011-02-16 8:55 ` Matthieu Moy
2011-02-16 9:42 ` Martin von Zweigbergk
2011-02-16 10:08 ` Jakub Narebski
2011-02-16 10:26 ` Matthieu Moy
2011-02-16 10:27 ` Sverre Rabbelier
2011-02-16 17:56 ` Junio C Hamano
2011-02-16 18:08 ` Sverre Rabbelier
2011-02-16 18:37 ` Junio C Hamano
2011-02-18 0:51 ` Martin von Zweigbergk
2011-02-18 0:57 ` Martin von Zweigbergk
2011-02-16 10:54 ` Bernhard R. Link
2011-02-14 9:40 ` [1.8.0] Provide proper remote ref namespaces Jakub Narebski
2011-02-14 15:45 ` Marc Branchaud
2011-02-14 19:35 ` Nicolas Pitre
2011-02-14 18:30 ` Junio C Hamano
2011-02-14 19:06 ` Nicolas Pitre
2011-02-14 19:46 ` Sverre Rabbelier
2011-02-07 6:54 ` Matthieu Moy
2011-02-07 0:14 ` Dmitry Potapov
2011-02-05 18:39 ` Nicolas Pitre
2011-02-05 19:37 ` Jeff King
2011-02-05 19:55 ` Nicolas Pitre
2011-02-05 23:39 ` Johan Herland
2011-02-06 20:04 ` Junio C Hamano
2011-02-07 16:10 ` Marc Branchaud
2011-02-07 19:40 ` Junio C Hamano
2011-02-07 20:37 ` Nicolas Pitre
2011-02-07 5:18 ` Jeff King
2011-02-07 14:53 ` Nicolas Pitre
2011-02-07 20:25 ` Jeff King
2011-02-07 20:51 ` Nicolas Pitre
2011-02-07 20:56 ` Jeff King
2011-02-01 23:15 ` [1.8.0] Remote tag namespace Johan Herland
-- strict thread matches above, loose matches on Subject: below --
2011-02-07 3:35 [1.8.0] Provide proper remote ref namespaces Johan Herland
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: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201102020322.00171.johan@herland.net \
--to=johan@herland.net \
--cc=git@vger.kernel.org \
--cc=nico@fluxnic.net \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
--cc=srabbelier@gmail.com \
/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://80x24.org/mirrors/git.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).