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
next prev parent 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).