All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Liam Girdwood <lrg@ti.com>,
	linux-kernel@vger.kernel.org, Junio C Hamano <gitster@pobox.com>,
	Git Mailing List <git@vger.kernel.org>
Subject: Re: Regulator updates for 3.3
Date: Tue, 10 Jan 2012 23:17:03 +0000	[thread overview]
Message-ID: <20120110231700.GA14242@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <CA+55aFxvQF=Bm4ae6euB_UO8otMCuN9Lv37Zn3TpE-L7JH3Kzw@mail.gmail.com>

On Tue, Jan 10, 2012 at 02:54:27PM -0800, Linus Torvalds wrote:
> On Tue, Jan 10, 2012 at 2:27 PM, Mark Brown

> > Especially in the cases where the lack of the bug fix breaks the new
> > code it sems sensible enough to want to do the merges so that the
> > history includes things that actually work.

> So I don't mind merges if they have a lear reason for existing.

OK, good - I figured that was the case but wanted to make sure as you
were stating things rather more strongly than that.

Just to warn you there's also a whole stack of similar merges going to
come in via the sound tree too due to the same workflow, I *could* try
to rebuild the history and ask Takashi to redo his tree using that but
there's a lot of history there and it'd be hard to figure out which of
the merges was actually important.  Is it OK to leave things as they are
for this release?

> So right now "git merge" (and "git pull") make it too easy to make
> those meaningless merge commits. If instead of seven pointless merges
> you had (say) had two merges that had messages about *why* they
> weren't pointless, I'd be perfectly happy.

> Addid junio and git to the cc just to bring up this issue of bad UI
> once again. I realize it could break old scripts to start up an editor
> window, but still..

I'd use a configuration option that popped up an editor by default, even
if I did have to manually enable it.

  reply	other threads:[~2012-01-10 23:17 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-09  7:37 Regulator updates for 3.3 Mark Brown
2012-01-10 18:27 ` Linus Torvalds
2012-01-10 18:45   ` Mark Brown
2012-01-10 19:18     ` Linus Torvalds
2012-01-10 22:27       ` Mark Brown
2012-01-10 22:54         ` Linus Torvalds
2012-01-10 23:17           ` Mark Brown [this message]
2012-01-11  2:28           ` Junio C Hamano
2012-01-11  2:47             ` Linus Torvalds
2012-01-11  3:03               ` Junio C Hamano
2012-01-11  3:14                 ` Linus Torvalds
2012-01-11  6:59                   ` Re* " Junio C Hamano
2012-01-11 16:14                     ` Phil Hord
2012-01-11 16:23                     ` Linus Torvalds
2012-01-16  0:14                     ` Pete Harlan
2012-01-16 23:33                       ` Junio C Hamano
2012-01-16 23:43                         ` Martin Fick
2012-01-17  5:33                         ` Pete Harlan
2012-01-17  6:13                           ` Junio C Hamano
2012-01-11  3:21                 ` Linus Torvalds
2012-01-11 18:40           ` Paul Gortmaker
2012-01-13 19:12             ` [PATCH] merge: Make merge strategy message follow the diffstat Junio C Hamano
2012-01-13 19:27               ` Nguyen Thai Ngoc Duy
2012-01-13 19:49                 ` Linus Torvalds
2012-01-17  8:03                   ` Miles Bader

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=20120110231700.GA14242@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lrg@ti.com \
    --cc=torvalds@linux-foundation.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.
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.