git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Mikael Magnusson <mikachu@gmail.com>,
	git@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: linux-x86-tip: pilot error?
Date: Mon, 23 Jun 2008 13:22:33 +0200	[thread overview]
Message-ID: <20080623112233.GB7485@elte.hu> (raw)
In-Reply-To: <20080623111144.GP22569@linux.vnet.ibm.com>


* Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:

> > That would be expected behavior, yes. You can try a "test-pull" into 
> > core/rcu:
> > 
> >   git-checkout -b test-rcu tip/core/rcu
> >   git-merge paulmck-rcu-2008-06-23   # replace with git-pull and an URI
> > 
> > ... and then look at how "git-log test-rcu..linus/master" looks like. It 
> > should show all the changes of the RCU topic, your two new commits 
> > included.
> 
> The git-merge seemed to run normally, but the git-log command showed
> no output.  Hmmm...

sorry, i accidentally reversed the arguments, that should have been:

  git-shortlog linus/master..test-rcu
  git-log linus/master..test-rcu

the way such things can be noticed is to replace git-log with git-diff:

   git-diff test-rcu..linus/master

and if that does not come empty (and it didnt), it's an ordering 
problem.

> >  $ git-describe 481c5346d0981940ee63037eb53e4e37b0735c10
> >  v2.6.26-rc7-25-g481c534
> 
> So this shows the last linus/master commit -not- containing the patch, 
> correct?  Ah, the most recent -tag-.  So I have to be a bit careful 
> about creating tags if I want this to work for me.  Fair enough!

you most definitely need to be careful about creating tags. Tags have 
extra security value (Linus signs them, etc.) and Git will not override 
an existing tag by default in any case. So you definitely dont want to 
create a tag that collides with any expected future upstream tag. (such 
as "v2.6.26" or "2.6.26-rc8")

tags are mostly meant for release management - our use of tags to save 
the "merge base" of topic branches in -tip can be considered mild abuse. 

( But we did not want to pullute the already crowded branch space with 
  extra technical branches. -tip has 127 branches at this moment, 89 of 
  which are topic branches - creating a base branch for each topic would 
  add another 89 branches and bring it all up to 216 branches. )

	Ingo

  reply	other threads:[~2008-06-23 11:23 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-22 12:36 linux-x86-tip: pilot error? Paul E. McKenney
2008-06-22 12:48 ` Mikael Magnusson
2008-06-22 13:21   ` Paul E. McKenney
2008-06-22 21:11     ` Mikael Magnusson
2008-06-22 21:42     ` Björn Steinbrink
2008-06-22 22:21       ` Paul E. McKenney
2008-06-23  7:14     ` Ingo Molnar
2008-06-23  9:57       ` Paul E. McKenney
2008-06-23 10:24         ` Ingo Molnar
2008-06-23 11:11           ` Paul E. McKenney
2008-06-23 11:22             ` Ingo Molnar [this message]
2008-06-23 16:14               ` Daniel Barkalow
2008-06-23 15:12       ` Jeff King
2008-06-23 15:31         ` Ingo Molnar
2008-06-23 16:05         ` Ingo Molnar

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=20080623112233.GB7485@elte.hu \
    --to=mingo@elte.hu \
    --cc=git@vger.kernel.org \
    --cc=mikachu@gmail.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=tglx@linutronix.de \
    /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;
as well as URLs for NNTP newsgroup(s).