all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Óscar Fuentes" <ofv@wanadoo.es>
To: emacs-devel@gnu.org
Subject: Re: Please don't use revision numbers on commit messages (and elsewhere).
Date: Fri, 01 Apr 2011 03:20:14 +0200	[thread overview]
Message-ID: <87k4fevkc1.fsf@wanadoo.es> (raw)
In-Reply-To: AANLkTimiuSOdOvkki9EGb7Nq43znjy8VHAZKJLE0ousH@mail.gmail.com

Juanma Barranquero <lekktu@gmail.com> writes:

[snip]

>> Across *any* branches,
>> including the "random" ones (whatever your definition of "random branch"
>> is.)
>
> My comment about "random branches" is an answer to your "(and this
> only after setting some options, as Emacs did.)".
>
> We're developing Emacs; it's irrelevant whether other projects do or
> do not set these options.

Anyone can setup a public repo anytime, anywhere. Let's think of a
long-lived feature branch of the type of lexbind or bidi which, for
whatever reason, the participating developers finds more convenient to
host outside of Savannah. Unless he remembers to set the appropriate
options on the public repo's config, it will be subjected to the same
line revision renumbering that happened to Emacs' trunk on the
past. Recommending to use revision ids instead of revision numbers would
help to avoid the problem.

> I don't foresee that super-distributed future that you imagine for
> Emacs. And if it does come to pass, it's everyone's responsibility to
> clearly label their revnos.
>
> After all, if you have a branch cloned from trunk, and you do
> development on it, and you do refer to revids in the changelogs, these
> revids won't be meaningful for the trunk's users either unless the
> branch is merged to the trunk. If you just send a patch, revids will
> be as meaningless as revnos would be.

In the case of patches, using revision ids on the commit messages is,
actually, most convenient, because on that case the referenced ids are
unambiguous no matter on which branch the patch is applied.

> The Emacs project does not usually maintain a large number of active
> branches.

On a distributed project, you don't know how many active branches exist
out there.

> And, with a shared repo, "cloning a number of branches"
> isn't that problematic, space- or time-wise.

Let me expand with an example based on my past* experience. I have a
number of heterogeneous machines (different OS, varying network
connectivity, etc) and on all of them I have Emacs running (of
course!). I've my private branch with some customizations, which is what
I use for building and installing Emacs on all those machines. Keeping
the private branch mirrored among all of them means work. Keeping
mirrors for `trunk', emacs-23 and what-not is too much of a burden (last
time I checked there was no simple & reliable method for synchronizing
sets of branches across multiple platforms.) In theory, having just my
private branch and merging trunk into it from time to time would be
enough. But then those commits messages referencing other revisions by
their numbers doesn't fit, as trunk's revision #110000 has another
number on my private branch.

(*) "That other DVCS" handled the described scenario perfectly.

> It's hard to envision you cloning emacs-23 to locate a revision, then
> removing it from the disk, then cloning it again to locate the next
> revision...
>
> Also, that example is currently quite artificial, because they aren't
> that many revnos in the ChangeLogs right now (I know, I checked the
> logs) and they overwhelming refer to the trunk. So you're describing a
> future, hypothetical problem.

Do you prefer to wait until the problem has manifested itself on all its
crudeness? :-)

[snip]




  reply	other threads:[~2011-04-01  1:20 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 [this message]
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
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=87k4fevkc1.fsf@wanadoo.es \
    --to=ofv@wanadoo.es \
    --cc=emacs-devel@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 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.