[Rpm-maint] RPM 4.9.0 alpha available

Michael Schroeder mls at suse.de
Thu Nov 25 12:11:23 UTC 2010


On Thu, Nov 25, 2010 at 12:55:17PM +0200, Panu Matilainen wrote:
> On Thu, 25 Nov 2010, Michael Schroeder wrote:
> 
> > On Thu, Nov 25, 2010 at 10:48:08AM +0200, Panu Matilainen wrote:
> > > If you have time to look at the more than one tilde-case, then please do. 
> > 
> > Okey, I'll send a patch later this day.
> 
> Cool.

Patch attached. Seems to work well, but a couple of testcases
in rpmvercmp.at would be nice.

Oh, I just noticed that it changes the semantics a bit:

old:
rpm.vercmp("1.", "1") -> 1
rpm.vercmp("1..", "1.") -> 0

new:
rpm.vercmp("1.", "1") -> 0
rpm.vercmp("1..", "1.") -> 0

That's because I changed the loop from "while (*one && *two)" to
"while (*one || *two)". The results are much saner with the change,
but we probably need to stay bug compatible. What do you think?

Cheers,
  Michael.

-- 
Michael Schroeder                                   mls at suse.de
SUSE LINUX Products GmbH, GF Markus Rex, HRB 16746 AG Nuernberg
main(_){while(_=~getchar())putchar(~_-1/(~(_|32)/13*2-11)*13);}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rpm-tilde.diff
Type: text/x-patch
Size: 845 bytes
Desc: not available
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20101125/ac190a57/attachment.bin>


More information about the Rpm-maint mailing list