Re: [PATCH] kernel.spec: let flavors/variants end with "+$flavor" in the uname

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 14.06.2013 16:33, Harald Hoyer wrote:
> On 06/14/2013 04:27 PM, Thorsten Leemhuis wrote:
>> Harald Hoyer wrote on 14.06.2013 16:02:
>>> On 06/13/2013 09:08 AM, harald@xxxxxxxxxx wrote:
>>>>
>>>> This patch changes the version of kernel flavor/variants to end
>>>> in "+$flavor" instead of ".$flavor". This makes it easier to detect
>>>> a flavor and a parser can separate it from the architecture.
>>>>
>>>> With that change we can correct kernel-install (of systemd) to call
>>>> new-kernel-package with --package kernel-$flavor, because the $flavor
>>>> can easily be extracted from the version string.
>>>> ---
>>>>
>>>> After talking to the rpm team, "+" is the only char left as a separator.
>>>>
>>>> Kernel compiles and runs:
>>>> $ uname -r
>>>> 3.10.0-0.rc5.git0.1.fc20.x86_64+debug
>>>> $ echo /lib/modules/*debug
>>>> /lib/modules/3.10.0-0.rc5.git0.1.fc20.x86_64+debug
>>>
>>> Any comments?
>>
>> I'll bite, even if I'm just a interested party. The + kind of looks ugly
>> to me. In your initial mail ("Patch for kernel variants/flavors") you wrote
> Come on... "+" instead of "-" or "." and it suddenly looks ugly to you?

Don't ask me why ;-) Maybe it's because it's unusual :)

> Actually, I kind of like it, because it springs right into your eyes :)

Or maybe that's actually what it is. Whatever, I don't care to deeply.

> IMHO, adding ".general" or ".default" actually has the potential for more
> breakage than changing "." into "+".

Definitely, but it makes the special (albeit default) case go away and
thus all these "%{?1:+%{1}}" and similar tricks in the specfile and
other tools can go away in the long run. That what makes me prefer this
solution.

Cu
knurd
_______________________________________________
kernel mailing list
kernel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/kernel





[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux