all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Karl Fogel <kfogel@red-bean.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: ofv@wanadoo.es, Stefan Monnier <monnier@iro.umontreal.ca>,
	emacs-devel@gnu.org
Subject: Re: merge conlict?
Date: Tue, 26 Jan 2010 10:15:51 -0500	[thread overview]
Message-ID: <87r5pc7v88.fsf@red-bean.com> (raw)
In-Reply-To: <E1NZhP2-00054J-Ff@fencepost.gnu.org> (Eli Zaretskii's message of "Tue, 26 Jan 2010 04:08:04 -0500")

Eli Zaretskii <eliz@gnu.org> writes:
>> My experience is indeed that Bzr helps me keep track of local changes
>> and work on long-time branches, but usually when it gets time to install
>> on the trunk, I really port the changes "by hand" rather than just ask
>> Bzr to merge some (set of) change(s), so that I can clean them up (and
>> often enough, update&improve them).  Many people use "rebase" for that
>> clean up.
>
>I agree with Óscar: this is a lot of potentially unnecessary work.
>
>Put yourself in my shoes: in the bidi branch I have roughly 2-3
>commits for each week since August 2009 (1-2 commits for merges from
>mainline, and 1 more for the bidi changes themselves I did during that
>weekend).  This sounds like not too much, but it accumulates over the
>months; do your math.  Having to sift through all that when the time
>comes to merge with the trunk is not my idea of efficient use of my
>scarce resources.  Especially since I don't see the boost in utility
>that would justify such an investment.  The need to clean up the
>ChangeLog entries is already a PITA, but that is at least marginally
>bearable (because most intermediate entries are simply deleted).
>
>Changes that do not require more than a single commit and take less
>than a day to develop and test I already make in the trunk directly,
>so "quickies" are out of this picture, and I'm talking only about
>feature branches that require several non-trivial commits over
>relatively prolonged periods of time (several days or weeks).
>
>Maybe people who can hack Emacs all day long can cope with this
>additional load.  I cannot, and I'm not convinced I'm the only one in
>this league.
>
>Switching to Bazaar was advertised as a boon for occasional
>contributors.  That is supposed to cover people who don't have a lot
>of time to invest in overhead activities.  Now it somehow turns out
>that developing with Bazaar is more work than it was with CVS?  So
>what did we switch for? to allow some advanced to exotic workflows for
>those who have unlimited time to begin with?  That simply doesn't make
>any sense to me.
>
>But in any case, until Someone(TM) writes up a proposal for merging
>feature branches that can be made part of the workflow, any
>intelligent and useful discussion of this issue is hardly possible,
>IMO.

I haven't been able to follow this entire thread, but: why doesn't the
recommended workflow suffice?

Is the problem that some people are saying that all the internal commits
on your local branch should be clean & buildable & lint-free, in
addition to the final merge being similarly clean?

If so, I think that would be an unreasonable expectation, and not one
I've seen other projects enforce.

The final merge needs to be of publishable quality, of course -- it's
going on the mainline.  But the commits that led up to it?  That's
really up to the committer(s).  No one else ever has to build those
commits, after all.  Even if the local branch is published somewhere
independently, so others can participate in its development, the
developer(s) can determine what quality standards they want to uphold
pre-merge.  The Emacs project as a whole should just uphold quality on
its official branches, especially the mainline.

Or if that's not what the problem is here, then sorry for the noise.  I
want to help, if I can, but I simply can't keep up with threads this
large, so it's possible my response will be mis-aimed.

-Karl




  parent reply	other threads:[~2010-01-26 15:15 UTC|newest]

Thread overview: 100+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1NZJ1t-00089o-ST@monty-python.gnu.org>
2010-01-25  7:43 ` merge conlict? Eli Zaretskii
2010-01-25  8:46   ` Glenn Morris
2010-01-25  9:29     ` Eli Zaretskii
2010-01-25 13:54       ` NEVER use `bzr push' for sending changes upstream (was: merge conlict?) Óscar Fuentes
2010-01-25 17:29         ` NEVER use `bzr push' for sending changes upstream Mark A. Hershberger
2010-01-25 18:28           ` Óscar Fuentes
2010-01-25  9:26   ` merge conlict? Andreas Schwab
2010-01-25 10:08     ` Eli Zaretskii
2010-01-25 10:19       ` Andreas Schwab
2010-01-25 10:30         ` Óscar Fuentes
2010-01-25 11:06           ` Andreas Schwab
2010-01-25 11:14             ` Eli Zaretskii
2010-01-25 11:27             ` Óscar Fuentes
2010-01-25 11:55               ` Teemu Likonen
2010-01-25 12:22                 ` Óscar Fuentes
2010-01-25 12:34                   ` Andreas Schwab
2010-01-25 12:48                     ` Óscar Fuentes
2010-01-25 13:00                       ` Andreas Schwab
2010-01-25 13:38                         ` Óscar Fuentes
2010-01-25 14:26                           ` Andreas Schwab
2010-01-25 14:39                             ` Óscar Fuentes
2010-01-25 15:47                               ` Andreas Schwab
2010-01-25 15:57                                 ` Óscar Fuentes
2010-01-25 16:10                                   ` Andreas Schwab
2010-01-25 18:06                                     ` Óscar Fuentes
2010-01-25 18:10                                       ` Andreas Schwab
2010-01-25 18:24                                         ` Óscar Fuentes
2010-01-25 18:31                                           ` Andreas Schwab
2010-01-25 18:50                                             ` Óscar Fuentes
2010-01-25 19:09                                               ` Andreas Schwab
2010-01-25 19:21                                                 ` Óscar Fuentes
2010-01-25 19:39                                                   ` Andreas Schwab
2010-01-25 20:07                                                     ` Óscar Fuentes
2010-01-25 20:56                                                       ` Andreas Schwab
2010-01-25 21:24                                                         ` Óscar Fuentes
2010-01-25 21:44                                                           ` Andreas Schwab
2010-01-25 23:19                                                             ` Óscar Fuentes
2010-01-25 23:54                                                               ` Andreas Schwab
2010-01-25 23:57                                                                 ` Óscar Fuentes
2010-01-26  2:02                                                               ` Stefan Monnier
2010-01-26  9:08                                                                 ` Eli Zaretskii
2010-01-26 15:11                                                                   ` Stefan Monnier
2010-01-26 18:11                                                                     ` Eli Zaretskii
2010-01-26 15:15                                                                   ` Karl Fogel [this message]
2010-01-26 17:13                                                                     ` Stephen J. Turnbull
2010-01-26 17:09                                                                       ` Karl Fogel
2010-01-26 18:14                                                                     ` Eli Zaretskii
2010-01-25 22:10                                                           ` David Reitter
2010-01-25 23:35                                                             ` Óscar Fuentes
2010-01-26  0:03                                                               ` Andreas Schwab
2010-01-26  1:38                                                             ` Stephen J. Turnbull
2010-01-26  4:30                                                               ` David Reitter
2010-01-25 13:41                   ` Teemu Likonen
2010-01-25 13:52                     ` Thierry Volpiatto
2010-01-25 12:30                 ` Óscar Fuentes
2010-01-26  0:05           ` Richard Stallman
2010-01-26  1:49             ` Stephen J. Turnbull
2010-01-26  4:25               ` Eli Zaretskii
2010-01-26  9:16                 ` Andreas Schwab
2010-01-26 10:31                   ` Eli Zaretskii
2010-01-26 10:51                     ` Andreas Schwab
2010-01-26 10:06               ` Óscar Fuentes
2010-01-26 16:49               ` Richard Stallman
2010-01-26 16:49               ` Richard Stallman
2010-01-26 17:50                 ` Eli Zaretskii
2010-01-26 18:20                   ` Lennart Borgman
2010-01-26 18:42                     ` Karl Fogel
2010-01-26 19:10                       ` Eli Zaretskii
2010-01-26 19:38                       ` Stephen J. Turnbull
2010-01-26 22:43                   ` Richard Stallman
2010-01-27  1:18                     ` Stefan Monnier
2010-01-27  4:08                       ` Eli Zaretskii
2010-01-25 10:30         ` Eli Zaretskii
2010-01-25 10:51           ` Andreas Schwab
2010-01-25 11:09             ` Eli Zaretskii
2010-01-25 12:36               ` Andreas Schwab
2010-01-25 12:43                 ` Óscar Fuentes
2010-01-25 10:27       ` Óscar Fuentes
2010-01-26  3:02     ` Stefan Monnier
2010-01-26  8:38       ` Eli Zaretskii
2010-01-26  9:29       ` Andreas Schwab
2010-01-26 10:10         ` Óscar Fuentes
2010-01-26 10:28           ` Andreas Schwab
2010-01-26 12:40           ` Andreas Schwab
2010-01-25 11:33   ` Teemu Likonen
2010-01-25 12:02     ` Eli Zaretskii
2010-01-25 12:38       ` Andreas Schwab
2010-01-25 13:17       ` Teemu Likonen
2010-01-25 14:22         ` Eli Zaretskii
2010-01-25 14:03     ` Interpreting git diff --cc [was: merge conlict?] Stephen J. Turnbull
2010-01-26  0:05     ` merge conlict? Richard Stallman
2010-01-26  0:04   ` Richard Stallman
2010-01-26  2:14     ` Stephen J. Turnbull
2010-01-26  4:26     ` Eli Zaretskii
2010-01-26 16:50       ` Richard Stallman
2010-01-26 17:53         ` Eli Zaretskii
2010-01-28  2:10   ` Katsumi Yamaoka
2010-01-28  4:03     ` Eli Zaretskii
2010-01-28  4:13       ` Katsumi Yamaoka
2010-01-28  5:41         ` 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

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

  git send-email \
    --in-reply-to=87r5pc7v88.fsf@red-bean.com \
    --to=kfogel@red-bean.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=ofv@wanadoo.es \
    /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.