diff --git a/pisi/history.py b/pisi/history.py index 5bc3c3c4..283fc553 100644 --- a/pisi/history.py +++ b/pisi/history.py @@ -29,10 +29,7 @@ class PackageInfo: a_build = [autoxml.String, autoxml.optional] def __str__(self): - if self.build: - return self.version + "-" + self.release + "-" + self.build - else: - return self.version + "-" + self.release + return self.version + "-" + self.release + "-" + (self.build or '?') class Repo: a_operation = [autoxml.String, autoxml.mandatory] diff --git a/pisi/operations/history.py b/pisi/operations/history.py index 006081fe..5ebd157a 100644 --- a/pisi/operations/history.py +++ b/pisi/operations/history.py @@ -10,7 +10,6 @@ # Please read the COPYING file. import os -import exceptions import gettext __trans = gettext.translation("pisi", fallback=True) _ = __trans.ugettext @@ -29,10 +28,11 @@ def __pkg_already_installed(name, pkginfo): if not installdb.has_package(name): return False - verno = str(pkginfo).split("-") - ver, rel = verno[0:2] - build = int(verno[2]) if len(verno) > 2 else None - + ver, rel, build = str(pkginfo).split("-") + if build == '?': + build = None + else: + build = int(build) return (ver, rel, build) == installdb.get_version(name) def __listactions(actions):