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(a)redhat.com 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