unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Eli Zaretskii <eliz@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Latest merge from the emacs-23 branch
Date: Sat, 18 Dec 2010 10:58:14 -0500	[thread overview]
Message-ID: <jwvr5dft6x4.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <83sjxwf18y.fsf@gnu.org> (Eli Zaretskii's message of "Sat, 18 Dec 2010 01:06:21 +0200")

>> Cherrypicking is unworkable since it means doing all the
>> history-tracking by hand.
> What do you mean by "history-tracking", and why do we need to do it by
> hand?

By history tracking I mean keeping track of what has been merged where
and when.  E.g. CVS did not do it and this is one of the main reasons
why I've pushed to change VCS.  When we want to add the content of one
branch to another, we need to know that history in order to know which
part of the branch still needs to be added.  As long as we do "whole
branch merges", Bazaar keeps track of it for us.  If we start doing it
piecewise, Bazaar doesn't know how to do it for us, so we'd have to do
it by hand.  We used to do it by hand with CVS, but we did it for "whole
branch merges", which is the easy case (so easy that even Bazaar can do
it for us)., doing it on a revision-by-revision basis is a lot
more troublesome.

>> I'm not sure what you mean by "some of them are not actually
>> included in the merge", tho.  AFAIK they are included in the sense
>> that the corresponding change is now present on the trunk.

> No, there actually seem to be 2 different revisions on the trunk now that
> appear to solve the same issue in two different ways.  For example, this
> "merge":

>  99634.2.670: Eli Zaretskii 2010-12-11 Fix bug #7398 with truncated glyphs

> is also present here:

>  102637: Eli Zaretskii 2010-12-11 Fix bug #7398 with truncated glyphs

Yup.  This is a non-issue.

> Reading such log messages will result in a lot of confusion, I'm
> afraid.

Time will tell if your fear is justified.


        Stefan



  reply	other threads:[~2010-12-18 15:58 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-17 16:03 Latest merge from the emacs-23 branch Eli Zaretskii
2010-12-17 18:19 ` Andreas Schwab
2010-12-17 19:22   ` Eli Zaretskii
2010-12-17 22:03 ` Stefan Monnier
2010-12-17 23:06   ` Eli Zaretskii
2010-12-18 15:58     ` Stefan Monnier [this message]
2010-12-18 17:37       ` Eli Zaretskii
2010-12-19 13:47         ` Stefan Monnier
2010-12-19 15:10           ` Eli Zaretskii

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=jwvr5dft6x4.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=eliz@gnu.org \
    --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 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).