mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Christoph Lameter <cl@linux-foundation.org>
Subject: Re: [GIT PULL] percpu changes for v3.14-rc1
Date: Tue, 21 Jan 2014 21:37:31 -0500	[thread overview]
Message-ID: <20140122023711.GA4860@htj.dyndns.org> (raw)
In-Reply-To: <CA+55aFy6k0ezh69eXGDAJPfKHDRLPurCUBKXW4pcV-UjQVGLmg@mail.gmail.com>

Hello, Linus.

On Tue, Jan 21, 2014 at 05:51:13PM -0800, Linus Torvalds wrote:
> On Tue, Jan 21, 2014 at 1:48 AM, Tejun Heo <tj@kernel.org> wrote:
> >
> > I messed up the for-3.14 branch (committed stuff to for-next) and had
> > to rebuild for-3.14 by cherry-picking; however, the result is the same
> > as published to the next tree through for-next.
> >
> > The changes are available in the following git branch
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/tj/percpu.git
> 
> You messed up the pull request too.. The branch name is missing from
> that git line, even if you did mention it a few lines earlier...

Oops, sorry.  The branch is for-3.14.

I have no idea how that happened tho.  That even isn't a part that I
edit.  I did

  git request-pull master git://git.kernel.org/pub/scm/linux/kernel/git/tj/percpu.git for-3.14 > out

and then pulled in that file and added the description on top and diff
at the end.  I still have the "out" file created by the above command
and it also lacks the branch tag, so it definitely wasn't me somehow
deleting it while editing.  If I run the git-request-pull again, it
does have "for-3.14" there with everything else identical.  I wonder
whether git-request-pull somehow skips over branch tag when remote
for-3.14 doesn't match local one?

Ooh, right, that was it.  So, after running git-request-pull for the
first time, I rebuilt for-3.14, did git push -f and then ran
git-request-pull.  At that point, the new for-3.14 hasn't propagated
to git://git.kernel.org yet, so git-request-pull couldn't find the
head which matched the SHA1 and thus omitted printing the branch.  I
wonder whether this is a new behavior.  I saw the warning message
multiple times but ISTR the generated pull request having the branch
name specified on the command line regardless.  Maybe it should just
fail rather than generating pull request w/o branch tag?

Thanks.

-- 
tejun

      reply	other threads:[~2014-01-22  2:37 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-21  9:48 Tejun Heo
2014-01-22  1:51 ` Linus Torvalds
2014-01-22  2:37   ` Tejun Heo [this message]

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=20140122023711.GA4860@htj.dyndns.org \
    --to=tj@kernel.org \
    --cc=cl@linux-foundation.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®