From: Stephen Rothwell <sfr@canb.auug.org.au>
To: Alex Williamson <alex.williamson@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
kvm <kvm@vger.kernel.org>,
fengguang.wu@intel.com, julia.lawall@lip6.fr,
jiang.liu@huawei.com
Subject: Re: [GIT PULL] VFIO updates for v3.8
Date: Wed, 12 Dec 2012 11:44:30 +1100 [thread overview]
Message-ID: <20121212114430.cceef241f595a95e16c48874@canb.auug.org.au> (raw)
In-Reply-To: <1355270816.3224.179.camel@bling.home>
[-- Attachment #1: Type: text/plain, Size: 1519 bytes --]
Hi Alex,
On Tue, 11 Dec 2012 17:06:56 -0700 Alex Williamson <alex.williamson@redhat.com> wrote:
>
> Is that a bad thing? I can start tagging from my next branch if that's
> preferred. Thanks,
Linus has said many times to not rebase before sending a pull request.
When you rebase your tree you effectively throw away your testing (since
the thing you rebased on top of may have introduced semantic conflicts
with the work in your tree). If you don't rebase your tested tree, any
conflicts are then restricted to the actual merge and can be fixed there
(or at least the diagnosis will lead there).
So, if I was a maintiner, at the start of the merge window (or just
before) I would create a test branch that contained my work plus a
*merge* with Linus' tree and do some testing on that and then ask Linus
to pull my tree (not the merged version). It may prove that the test
merge with Linus' tree produces an "interesting" syntactic conflict - in
this case I would mention that to Linus and put the merged tree somewhere
public for him to use as a guide. (Mind you, this conflict would already
have most likely been noted by the linux-next maintainer.)
Also, your testing may have brought to light a semantic conflict, in
which case the fix could be supplied to Linus with the pull request, or a
well changed logged back merge of Linus' tree containing the fix could be
done and Linus asked to pull the result.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2012-12-12 0:44 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-11 19:08 Alex Williamson
2012-12-11 23:46 ` Stephen Rothwell
2012-12-12 0:06 ` Alex Williamson
2012-12-12 0:44 ` Stephen Rothwell [this message]
2012-12-12 2:59 ` Alex Williamson
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=20121212114430.cceef241f595a95e16c48874@canb.auug.org.au \
--to=sfr@canb.auug.org.au \
--cc=alex.williamson@redhat.com \
--cc=fengguang.wu@intel.com \
--cc=jiang.liu@huawei.com \
--cc=julia.lawall@lip6.fr \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®