mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Stephen Rothwell <sfr@canb.auug.org.au>
To: Sage Weil <sage@newdream.net>
Cc: "\"Jörn Engel\"" <joern@logfs.org>,
	"Al Viro" <viro@ZenIV.linux.org.uk>,
	"Christoph Hellwig" <hch@lst.de>,
	linux-next@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: linux-next: merge fixups relative to the vfs tree
Date: Tue, 19 Jan 2010 15:19:55 +1100	[thread overview]
Message-ID: <20100119151955.1d07aa71.sfr@canb.auug.org.au> (raw)
In-Reply-To: <Pine.LNX.4.64.1001181512340.1759@cobra.newdream.net>

[-- Attachment #1: Type: text/plain, Size: 1384 bytes --]

Hi Sage,

On Mon, 18 Jan 2010 15:16:39 -0800 (PST) Sage Weil <sage@newdream.net> wrote:
>
> That works, thanks.  I was planning on just making sure my tree got pulled 
> after the vfs tree, but this will work either way.

It looks like you added my patch to your tree, but did not merge Al's
write_inode branch.  So, I assume that your tree is currently broken if
built on its own.

> More generally, I'm not sure I understand how your fixups are supposed to 
> be used down the line.  Is whichever tree that caused the conflict to get 
> merged second supposed to pick it up in their branch to pull, or are those 
> patches being fed to Linus somehow, or does he usually fix up those 
> conflicts himself?

Yeah, either the second tree's owner will do the fixup after the first is
merged into Linus' tree or (more commonly for simple fixups), Linus will
do the fix when he merges the second tree.  In this particular case, the
conflicts are not noticed by git during the merge, so they need to be
done manually by the second tree's owner, or by Linus (if we tell him).

I will attempt to remember to warn Linus if these conflicts remain during
the next merge window but if you and Jörn merge Al's branch and apply the
fix, then it will all be OK.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2010-01-19  4:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-18  7:28 Stephen Rothwell
2010-01-18 23:16 ` Sage Weil
2010-01-19  4:19   ` Stephen Rothwell [this message]
2010-01-19  4:47     ` Sage Weil

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=20100119151955.1d07aa71.sfr@canb.auug.org.au \
    --to=sfr@canb.auug.org.au \
    --cc=hch@lst.de \
    --cc=joern@logfs.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=sage@newdream.net \
    --cc=viro@ZenIV.linux.org.uk \
    /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®