2010-07-16 18:18:16

by Pete Zaitcev

[permalink] [raw]
Subject: I did a git pull and a plus appeared

Dear All:

I have a tree that existed for a few years (a clone of Linus' tree).
Today I did a git pull in it, and a plus appeared in the built version.
E.g. after "make modules_install":

[root@niphredil linux-2.6-tip]# ls /lib/modules/
2.6.33.1-19.fc13.x86_64 2.6.34-rc6-tip 2.6.35-rc1-tip
2.6.34-43.fc14.x86_64 2.6.35-0.36.rc4.git5.fc14.x86_64 2.6.35-rc5-tip+
[root@niphredil linux-2.6-tip]#

How do I get rid of the plus? I have (had for a long time):

[zaitcev@niphredil linux-2.6-tip]$ grep LOCAL .config
CONFIG_LOCALVERSION="-tip"
# CONFIG_LOCALVERSION_AUTO is not set
CONFIG_X86_LOCAL_APIC=y
[zaitcev@niphredil linux-2.6-tip]$

Thanks,
-- Pete


2010-07-16 20:02:39

by Jeff Garzik

[permalink] [raw]
Subject: Re: I did a git pull and a plus appeared

On 07/16/2010 02:18 PM, Pete Zaitcev wrote:
> Dear All:
>
> I have a tree that existed for a few years (a clone of Linus' tree).
> Today I did a git pull in it, and a plus appeared in the built version.
> E.g. after "make modules_install":
>
> [root@niphredil linux-2.6-tip]# ls /lib/modules/
> 2.6.33.1-19.fc13.x86_64 2.6.34-rc6-tip 2.6.35-rc1-tip
> 2.6.34-43.fc14.x86_64 2.6.35-0.36.rc4.git5.fc14.x86_64 2.6.35-rc5-tip+
> [root@niphredil linux-2.6-tip]#
>
> How do I get rid of the plus? I have (had for a long time):
>
> [zaitcev@niphredil linux-2.6-tip]$ grep LOCAL .config
> CONFIG_LOCALVERSION="-tip"
> # CONFIG_LOCALVERSION_AUTO is not set
> CONFIG_X86_LOCAL_APIC=y
> [zaitcev@niphredil linux-2.6-tip]$

Is the '+' listed at the top of the toplevel Makefile?

Jeff


2010-07-16 20:15:08

by David Miller

[permalink] [raw]
Subject: Re: I did a git pull and a plus appeared

From: Jeff Garzik <[email protected]>
Date: Fri, 16 Jul 2010 16:02:25 -0400

> On 07/16/2010 02:18 PM, Pete Zaitcev wrote:
>> How do I get rid of the plus? I have (had for a long time):
>>
>> [zaitcev@niphredil linux-2.6-tip]$ grep LOCAL .config
>> CONFIG_LOCALVERSION="-tip"
>> # CONFIG_LOCALVERSION_AUTO is not set
>> CONFIG_X86_LOCAL_APIC=y
>> [zaitcev@niphredil linux-2.6-tip]$
>
> Is the '+' listed at the top of the toplevel Makefile?

It's being put there by Kbuild, this has been discussed quite a bit
over the past few weeks I think.

The "+" comes from scripts/setlocalversion

2010-07-17 00:26:32

by Pete Zaitcev

[permalink] [raw]
Subject: Re: I did a git pull and a plus appeared

On Fri, 16 Jul 2010 13:15:21 -0700 (PDT)
David Miller <[email protected]> wrote:

> It's being put there by Kbuild, this has been discussed quite a bit
> over the past few weeks I think.

I suspected something like that, but finding the relevant discussion
afterwards is different.

> The "+" comes from scripts/setlocalversion

Right, found that quickly, but the question is what the authors' intent was.
I cannot tell from the code. Folks at #kernel seem to think the plus means
"built from a tarball", in which case this is a bug (the kernel was built
from a cloned tree).

I can easily add '+' to the list of filename characters in my terminal,
just thought to ask in case it's a bug.

-- Pete

2010-07-17 20:04:42

by David Rientjes

[permalink] [raw]
Subject: Re: I did a git pull and a plus appeared

On Fri, 16 Jul 2010, Pete Zaitcev wrote:

> Dear All:
>
> I have a tree that existed for a few years (a clone of Linus' tree).
> Today I did a git pull in it, and a plus appeared in the built version.
> E.g. after "make modules_install":
>
> [root@niphredil linux-2.6-tip]# ls /lib/modules/
> 2.6.33.1-19.fc13.x86_64 2.6.34-rc6-tip 2.6.35-rc1-tip
> 2.6.34-43.fc14.x86_64 2.6.35-0.36.rc4.git5.fc14.x86_64 2.6.35-rc5-tip+
> [root@niphredil linux-2.6-tip]#
>
> How do I get rid of the plus? I have (had for a long time):
>
> [zaitcev@niphredil linux-2.6-tip]$ grep LOCAL .config
> CONFIG_LOCALVERSION="-tip"
> # CONFIG_LOCALVERSION_AUTO is not set
> CONFIG_X86_LOCAL_APIC=y
> [zaitcev@niphredil linux-2.6-tip]$
>

I initially added the `+' to the version string in 85a256d8 during this
release cycle. As the changelog indicates, it means that your tree is
beyond a tagged commit and LOCALVERSION= was not passed on the command
line so the tree must be described as a modified 2.6.35-rc5 kernel, for
example, instead of vanilla. That's the semantics that the `+' indicates.

2010-07-17 20:15:07

by Nico Schottelius

[permalink] [raw]
Subject: Re: I did a git pull and a plus appeared

Hey guys,

Pete Zaitcev [Fri, Jul 16, 2010 at 06:26:09PM -0600]:
> > The "+" comes from scripts/setlocalversion
>
> Right, found that quickly, but the question is what the authors' intent was.
> I cannot tell from the code. Folks at #kernel seem to think the plus means
> "built from a tarball", in which case this is a bug (the kernel was built
> from a cloned tree).
>
> I can easily add '+' to the list of filename characters in my terminal,
> just thought to ask in case it's a bug.

I'm soon on a long trip, has anybody fixed it already?

Otherwise I'm having a look at it.

Cheers,

Nico

--
New PGP key: 7ED9 F7D3 6B10 81D7 0EC5 5C09 D7DC C8E4 3187 7DF0
Please resign, if you signed 9885188C or 8D0E27A4.

Currently moving *.schottelius.org to http://www.nico.schottelius.org/ ...


Attachments:
(No filename) (792.00 B)
(No filename) (198.00 B)
Download all attachments