Re: [PATCH] backports: fix high RC version parsing in dependency library

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

 



On 04/28/2015 02:25 PM, Eliad Peller wrote:
> On Tue, Apr 28, 2015 at 1:26 PM, Mathy Vanhoef <vanhoefm@xxxxxxxxx> wrote:
>> On Tue, Apr 28, 2015 at 10:27 AM, Arend van Spriel <arend@xxxxxxxxxxxx> wrote:
>>> On 04/28/15 01:56, Mathy Vanhoef wrote:
>>>>
>>>> This updates the dependency library to handle RC versions up to 39. For
>>>> example, coccinelle version "1.0.0" is now correctly treated as being
>>>> higher
>>>> than version "1.0.0-rc24".
>>>
>>>
>>> This one is duplicate to the patch Eliad submitted [1]. Any reason for
>>> choosing rc 40?
>>
>> I missed that patched. My reasoning was that previously at least the Linux
>> kernel went up to rc 39 with 2.6.39. So 40 should be a good limit to support
>> most cases.
>>
> note that 39 in this case is the "sublevel", not the "extraversion".
> 
> Eliad.

I will take your patch Eliad. There are so many magic numbers in this
code it probably has to get cleaned up, a warning for such a case like
we have it now would also be nice.

Hauke

--
To unsubscribe from this list: send the line "unsubscribe backports" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux