all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Stephen J. Turnbull" <stephen@xemacs.org>
To: Uday S Reddy <u.s.reddy@cs.bham.ac.uk>
Cc: emacs-devel@gnu.org
Subject: Re: Please don't use revision numbers on commit messages (and elsewhere).
Date: Sat, 02 Apr 2011 00:00:23 +0900	[thread overview]
Message-ID: <87pqp6m2yg.fsf@uwakimon.sk.tsukuba.ac.jp> (raw)
In-Reply-To: <in47no$g8v$1@dough.gmane.org>

Uday S Reddy writes:

 > I am still trying to understand how bad this problem is.

It's this bad:

Right now, you and I simultaneously clone the mainline at r42.  You
commit, creating r43, because as far as bzr knows, you're on the
mainline.  Oops, you missed something, and you commit your r44, with
the message "Fix bug introduced in r43."  While you're fixing the bug,
I commit and push, and my commit becomes mainline r43.  Now when you
pull and merge, your commits that used to be on the (local) mainline
with revnos r43 and r44 are now off the mainline and have revnos
r42.1.1 and r42.1.2, and the commit message is now a dastardly attack
on my reputation. ;-)

Yes, if something is already in the master repo, you can refer to it
by revno, as long as you never intend to push it anywhere else, or
allow others to pull from your branch (you can't be sure that they
have the same history you do).  But you can't refer to your own
commits by revno until they've been pushed to the master.




  reply	other threads:[~2011-04-01 15:00 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-31 20:47 Please don't use revision numbers on commit messages (and elsewhere) Óscar Fuentes
2011-03-31 21:36 ` Lennart Borgman
2011-03-31 21:53   ` Óscar Fuentes
2011-03-31 21:59     ` Lennart Borgman
2011-03-31 22:06       ` Óscar Fuentes
2011-03-31 22:18         ` Lennart Borgman
2011-03-31 22:58       ` Juanma Barranquero
2011-04-01  7:42   ` Eli Zaretskii
2011-04-01  7:58     ` Andreas Schwab
2011-04-01  8:02       ` Eli Zaretskii
2011-04-01  8:17         ` Andreas Schwab
2011-04-01  8:42           ` Eli Zaretskii
2011-04-01  8:54             ` Andreas Schwab
2011-04-01 10:11               ` Eli Zaretskii
2011-04-01 10:21                 ` Andreas Schwab
2011-04-01 10:48                   ` Eli Zaretskii
2011-04-01 11:18                     ` Andreas Schwab
2011-04-01 13:15                       ` Eli Zaretskii
2011-04-01 13:32                         ` Andreas Schwab
2011-04-01 13:47                           ` Eli Zaretskii
2011-04-01 13:51                             ` Deniz Dogan
2011-04-01 15:26         ` Óscar Fuentes
2011-04-01 19:13           ` Eli Zaretskii
2011-04-01 20:17             ` Óscar Fuentes
2011-03-31 23:14 ` Juanma Barranquero
2011-04-01  0:11   ` Óscar Fuentes
2011-04-01  0:28     ` Juanma Barranquero
2011-04-01  1:20       ` Óscar Fuentes
2011-04-01  8:18         ` Eli Zaretskii
2011-04-01 12:08           ` David Kastrup
2011-04-01 13:15             ` Eli Zaretskii
2011-04-01 15:35           ` Óscar Fuentes
2011-04-01 19:52             ` Eli Zaretskii
2011-04-01 20:04               ` David Kastrup
2011-04-01 20:36                 ` Eli Zaretskii
2011-04-01 20:43               ` Óscar Fuentes
2011-04-01 10:34         ` Juanma Barranquero
2011-04-01 15:55           ` Óscar Fuentes
2011-04-01 21:53             ` Juanma Barranquero
2011-04-04 16:32           ` Nils Ackermann
2011-04-04 21:27             ` Juanma Barranquero
2011-04-04 21:36               ` Lennart Borgman
2011-04-04 21:49                 ` Juanma Barranquero
2011-04-04 22:03                   ` Lennart Borgman
2011-04-04 22:09                     ` Juanma Barranquero
2011-04-04 22:27               ` Stefan Monnier
2011-04-04 22:35                 ` Juanma Barranquero
2011-04-05 21:00                   ` Thien-Thi Nguyen
2011-04-05 21:00                   ` Thien-Thi Nguyen
2011-04-06  1:30                     ` Stefan Monnier
2011-04-06  2:55                       ` Stephen J. Turnbull
2011-04-06 12:47                         ` Thien-Thi Nguyen
2011-04-01  1:59       ` Stephen J. Turnbull
2011-04-01 10:00         ` Uday S Reddy
2011-04-01 15:00           ` Stephen J. Turnbull [this message]
2011-04-01 16:38             ` Uday S Reddy
2011-04-01 18:08               ` Stephen J. Turnbull
2011-04-01 18:56                 ` Uday S Reddy
2011-04-01 20:49                   ` Stephen J. Turnbull
2011-04-01 10:45         ` Juanma Barranquero
2011-04-01 14:51           ` Stefan Monnier
2011-04-01 15:14             ` Ted Zlatanov
2011-04-01 19:58             ` Juanma Barranquero
2011-04-02  2:12     ` Chong Yidong
2011-04-01  1:35   ` Stephen J. Turnbull
2011-04-01 10:39     ` Juanma Barranquero
2011-03-31 23:16 ` Thien-Thi Nguyen
2011-04-01  0:20   ` Óscar Fuentes
2011-04-01  8:38     ` Thien-Thi Nguyen
2011-04-01  9:36       ` Eli Zaretskii
2011-04-01 10:14         ` Eli Zaretskii
2011-04-01 17:38           ` Thien-Thi Nguyen
2011-04-01 15:38         ` Óscar Fuentes
2011-04-01 19:12           ` Eli Zaretskii
2011-04-01 20:21             ` Óscar Fuentes
2011-04-01 20:38               ` Eli Zaretskii
2011-04-01 21:40               ` Uday S Reddy
2011-04-02  0:03                 ` Óscar Fuentes
2011-04-02  6:20                   ` Uday S Reddy
2011-04-02 13:47                     ` Óscar Fuentes
2011-04-03  8:00                       ` Uday S Reddy
2011-04-03 16:13                         ` Óscar Fuentes
2011-04-04  9:29                           ` Uday S Reddy
2011-04-05  2:20                             ` Stephen J. Turnbull
2011-04-02  2:57                 ` Stephen J. Turnbull
2011-04-01  0:55 ` Stefan Monnier

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

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

  git send-email \
    --in-reply-to=87pqp6m2yg.fsf@uwakimon.sk.tsukuba.ac.jp \
    --to=stephen@xemacs.org \
    --cc=emacs-devel@gnu.org \
    --cc=u.s.reddy@cs.bham.ac.uk \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.