all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Stephen J. Turnbull" <stephen@xemacs.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: cyd@gnu.org, monnier@iro.umontreal.ca, emacs-devel@gnu.org
Subject: Re: Moving Emacs 24.3 development to emacs-24 branch soon
Date: Mon, 05 Nov 2012 19:39:59 +0900	[thread overview]
Message-ID: <87k3u0uydc.fsf@uwakimon.sk.tsukuba.ac.jp> (raw)
In-Reply-To: <83hap546rf.fsf@gnu.org>

Eli Zaretskii writes:

 > Actually, I find the graph of emacs-24 to be confusingly obscure.  See
 > the attached screenshots of "bzr qlog", where simple merges from (the
 > old) emacs-24 are now displayed as something utterly weird, and
 > backports from trunk look weirder still.

I don't understand what you find weird or "not simple" about these.
They indicate quite clearly what's happened.  In both cases the theme
is "work on a branch, test, and only when happy merge to mainline."
Yidong made a separate branch for the purpose of merging two public
branches.  Ken'ichi made a "feature" branch to develop some fixes.  If
you don't find that information useful, ignore it.  (Eg, use bzr log
--omit-merges.)  Is there some sense that it gets in your way?

(I had already written the following, but really the above summary is
what's relevant.  But if you're curious about how I interpret the
DAGs, read on.)

In your first screenshot, it seems that at r110121 Yidong simply made
a branch to ensure that any merge screwups didn't make it to trunk.
He then merged emacs-24 into the temporary branch at r110121.1.  He
made no mistakes and upon inspection the merge looked correct (ie, I
suppose he did a test build etc., and inspected the commit log to make
sure no emacs-24-only commits were added (from the word "backport" I
wonder about 107781.350, but I don't have all the context of that
patch).  So he merged the temporary branch back into trunk without
making any new commits himself.  It's possible that he had to fix
merge conflicts, but I would assume if he did it would be noted in the
log.

In the second one, the yellow branch created at r110016 appears to be
a "miscellaneous fix" branch that Ken'ichi is using as a testing
workspace or similar.  So he regularly merges up to trunk head, does
more work, and remerges from the branch to trunk.

I agree the duplicate commit messages do look weird, but every one of
them actually corresponds to a single commit on the branch, then
merged back to the trunk.  It is considered best practice in bzr to
provide a summary of the branch being merged so that bzr log -n1 gives
a "big picture" of the work being merged.  Since Ken'ichi is
committing a single change then merging, I think simply copying the
message makes sense.  If that bugs people, then Emacs could make a
style guide for commit messages that says "If you have multiple
commits, summarize in the merge log.  If there's only one, you can
copy that message, but prepend '(merge) '."  Or something like that.







  parent reply	other threads:[~2012-11-05 10:39 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-01  2:12 Moving Emacs 24.3 development to emacs-24 branch soon Chong Yidong
2012-11-01  6:57 ` Andreas Schwab
2012-11-01  9:30   ` Dani Moncayo
2012-11-01 13:04     ` Stefan Monnier
2012-11-01 10:09   ` Chong Yidong
2012-11-01 10:39     ` Dani Moncayo
2012-11-01 13:18 ` Andreas Schwab
2012-11-01 13:47 ` Eli Zaretskii
2012-11-01 13:58   ` Chong Yidong
2012-11-01 14:05     ` Andreas Schwab
2012-11-01 16:32     ` Stephen J. Turnbull
2012-11-01 17:22       ` Stefan Monnier
2012-11-03 12:28         ` Stephen J. Turnbull
2012-11-04 17:28           ` Eli Zaretskii
2012-11-04 17:39             ` Juanma Barranquero
2012-11-05 10:39             ` Stephen J. Turnbull [this message]
2012-11-01 16:42 ` Glenn Morris

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=87k3u0uydc.fsf@uwakimon.sk.tsukuba.ac.jp \
    --to=stephen@xemacs.org \
    --cc=cyd@gnu.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.