On Mon, Aug 27, 2007 at 07:15:07PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> On Saturday, 25 August 2007 at 01:01, Axel Thimm wrote:
> > On Fri, Aug 24, 2007 at 11:44:36PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> > > > I think rpm5 in Fedora is dangerous. At the very least it reverses
> > > > the ordering of letters and digits and thus breaks a ton of packaging
> > > > techniques. Any *-1.fc8 -> *.1.1.fc8 upgrade path is busted for
> > > > example.
> > >
> > > Are you 100% sure? I've seen Jeff deny this on IRC.
> >
> > Index: rpmvercmp.c
> > ===================================================================
> > RCS file: /cvs/devel/rpm/lib/rpmvercmp.c,v
> > retrieving revision 2.8.2.2
> > diff -u -b -B -w -p -r2.8.2.2 rpmvercmp.c
> > --- rpmvercmp.c 27 Aug 2006 12:58:19 -0000 2.8.2.2
> > +++ rpmvercmp.c 26 Apr 2007 22:36:26 -0000
> > @@ -76,7 +76,7 @@ int _rpmvercmp(const char * a, const cha
> > /* different types: one numeric, the other alpha (i.e.
> > empty) */
> > /* numeric segments are always newer than alpha segments */
> > /* XXX See patch #60884 (and details) from bugzilla
> > #50977. */
> > - if (two == str2) return (isnum ? 1 : -1);
> > + if (two == str2) return (isnum ? -1 : 1);
> > if (isnum) {
> > /* this used to be done by converting the digit segments
> > */
>
> The code present in current CVS HEAD is a bit different. Also I can't find
> this change in rpm5.org's CVS. Which revision were you comparing to 2.8.2.2?
I copy and pasted this from an original mail to rpm-devel by rpm5's
developer. Any later refactoring didn't undo the change AFAIK.
--
Axel.Thimm at ATrpms.net
Attachment:
pgpvNlyddCLYz.pgp
Description: PGP signature
_______________________________________________ fedora-advisory-board mailing list fedora-advisory-board@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-advisory-board
