[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-issues
Subject:    [oe-issues] [Bug 2061] New: Fix broken version comparison in ipkg-make-index
From:       bugzilla-daemon () tinman ! treke ! net (bugzilla-daemon at tinman ! treke ! net)
Date:       2007-03-29 12:42:57
Message-ID: 20070329124257.D8DFE401B33 () tinman ! treke ! net
[Download RAW message or body]

http://bugs.openembedded.org/show_bug.cgi?id=2061

           Summary: Fix broken version comparison in ipkg-make-index
           Product: Openembedded
           Version: Angstrom
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P1
         Component: Build
        AssignedTo: openembedded-issues at lists.openembedded.org
        ReportedBy: pmiscml at gmail.com
         QAContact: tinderbox-oe at gmx.net


For long and many people hit ipkg-make-index's breakage when it miscompared
version of packages, and sent newly packages to morgue, instead leaving old in
deploy. This was especially 100% reproducible when build SCM-based version
after build release version of some package.

For example, trying to build something with version of "1.2.2+cvs20070308-r0"
after having built its version of "1.2.2-r0", caused newer package to go
morgue.

Reason of this is simple: ipkg-make-index does "flat" version comparison, not
taking into account structural components of the version ID. So, after
comparing "1.2.2" prefix and skipping punctuation, it literally compared "c"
and "r", with expected outcomes.

Solution is to treat version string as consisting of components, and performing
comparison component by component.

Attached patch implements that.


-- 
Configure bugmail: http://bugs.openembedded.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic