summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMart Raudsepp <leio@gentoo.org>2009-09-06 18:53:36 +0300
committerMart Raudsepp <leio@gentoo.org>2009-09-06 18:58:29 +0300
commitbc2a76ef68f6979e30762b0cd05c4b7853af6bd9 (patch)
treefc2c5078a858e40396713dc8317c3b7eb043f239
parentFix crash happening when latest_packages doesn't have all entries for package... (diff)
downloadgentoo-bumpchecker-bc2a76ef68f6979e30762b0cd05c4b7853af6bd9.tar.gz
gentoo-bumpchecker-bc2a76ef68f6979e30762b0cd05c4b7853af6bd9.tar.bz2
gentoo-bumpchecker-bc2a76ef68f6979e30762b0cd05c4b7853af6bd9.zip
Fix crash happening when upstream releng version list has a nonsensical version for a pkg.
In 2.27.91 the craze of using nonsensical version strings continues (before we had a gucharmap-2.27.0-git) in the form of a gnome-media-2.27.90.fix release, which also happened to end up in releng release versions list for some reason. If a package parsing fails, don't append it to the list.
-rw-r--r--modules/gnome_module.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/modules/gnome_module.py b/modules/gnome_module.py
index 5a518cf..c9d498b 100644
--- a/modules/gnome_module.py
+++ b/modules/gnome_module.py
@@ -49,7 +49,11 @@ class GNOME:
# Ignore perl packages that don't have versions known in version-stable due to different infrastructure for hosting
# We have a workaround in compare_packages now, but we don't have a proper mapping to ruby-g* stuff yet, so ignore them for now
if not(components[0] == 'bindings' and components[3] == 'perl'):
- ret.append(package_module.Package(components[1] + "-" + components[2]))
+ pkg = package_module.Package(components[1] + "-" + components[2])
+ if (pkg.name and pkg.version):
+ ret.append(pkg)
+ else:
+ print "Warning: Ignoring package %s because parsing of its name or version string '%s' failed" % (components[1], components[2])
f.close()
return ret