unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Joshua Judson Rosen <rozzin@geekspace.com>
To: Andreas Schwab <schwab@linux-m68k.org>
Cc: esr@thyrsus.com, handa@gnu.org, Eli Zaretskii <eliz@gnu.org>,
	emacs-devel@gnu.org
Subject: Re: Policy on referencing/abbreviating git commit-IDs, going forward?
Date: Sun, 02 Mar 2014 16:18:49 -0500	[thread overview]
Message-ID: <8738j0ti2u.fsf@slice.rozzin.com> (raw)
In-Reply-To: <87sir0gx7y.fsf@igel.home> (Andreas Schwab's message of "Sun, 02 Mar 2014 21:30:09 +0100")

Andreas Schwab <schwab@linux-m68k.org> writes:
>
> Joshua Judson Rosen <rozzin@geekspace.com> writes:
>
> > abbreviation ambiguous, how will you trace out the now-ambiguously-
> > abbreviated commit? e.g.: up through July 29 2012, "7a62fbf"
> > unambiguously identified a commit from October 24 2007; on July 30 2012,
> > "7a62fbf" stopped being unambiguous.
>
> If you know it was previsouly unambiguous you know that it refers to the
> older one.

I don't think one necessarily _does_ know that in any meaningful way,
when the situation crops up analogously to how it just did; I'm just
thinking, it may be worth putting a "do not use abbreviated sha1 refs in
your commit comments" or similar policy statement somewhere prominent in
the wiki and other docs, if it's not already there.

When you find an abbreviated ref in a revision that got merged in
from another branch/repository (which is what just happened), you only
know that it was previously unambiguous _in the author's own local
repository_ where the text using the abbreviation was written, not
that it was previously unambiguous in the mainline repository, and
certainly not that it was previously unambiguous in the mainline
repository's trunk branch (or in any other particular branch).

Not everyone pulls/pushes all branches when they sync with the mainline
repository, so it's entirely possible that two developers working on
separate branches to not have each others ambiguously-abbreviated
revisions in their own repositories (even when one of those branches is
the trunk). I'm not sure what the probability is over however long git
will be in use; the probability surely increases if more developers
more-abbreviated refs, though.

-- 
"'tis an ill wind that blows no minds."



  reply	other threads:[~2014-03-02 21:18 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-01 17:13 Can anyone correct the Bazaar reference "revno:111954.1.97"? Eric S. Raymond
2014-03-01 18:24 ` Eli Zaretskii
2014-03-01 18:43   ` Eric S. Raymond
2014-03-01 21:35   ` Joshua Judson Rosen
2014-03-01 22:02     ` Eric S. Raymond
2014-03-02  3:47       ` Joshua Judson Rosen
2014-03-02 17:44         ` Eli Zaretskii
2014-03-02 18:26           ` Andreas Schwab
2014-03-02 20:01             ` Policy on referencing/abbreviating git commit-IDs, going forward? (was: Can anyone correct the Bazaar reference "revno:111954.1.97"?) Joshua Judson Rosen
2014-03-02 20:30               ` Policy on referencing/abbreviating git commit-IDs, going forward? Andreas Schwab
2014-03-02 21:18                 ` Joshua Judson Rosen [this message]
2014-03-02 20:35             ` Can anyone correct the Bazaar reference "revno:111954.1.97"? Eli Zaretskii
2014-03-02 21:08               ` Andreas Schwab
2014-03-02  3:53       ` Eli Zaretskii
2014-03-02  3:55     ` Eli Zaretskii
2014-03-02  5:53       ` Joshua Judson Rosen
2014-03-02 17:42         ` Eli Zaretskii
2014-03-02 18:25           ` Andreas Schwab
2014-03-02 20:33             ` Eli Zaretskii
2014-03-02 21:10               ` Andreas Schwab
2014-03-02 21:27                 ` Eli Zaretskii
2014-03-02 22:14                   ` Andreas Schwab
2014-03-03  3:35                     ` Eli Zaretskii
2014-03-03  3:40                       ` Eric S. Raymond
2014-03-03  5:35                         ` Stephen J. Turnbull
2014-03-03  5:49                           ` Eric S. Raymond
2014-03-02 19:08           ` Joshua Judson Rosen
2014-03-02 20:38             ` Eli Zaretskii
2014-03-02 22:00               ` Joshua Judson Rosen
2014-03-02 22:15                 ` Andreas Schwab
2014-03-02 22:22                   ` Joshua Judson Rosen

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=8738j0ti2u.fsf@slice.rozzin.com \
    --to=rozzin@geekspace.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=esr@thyrsus.com \
    --cc=handa@gnu.org \
    --cc=schwab@linux-m68k.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).