diff options
author | Chris Lamb <lamby@debian.org> | 2016-08-04 17:49:15 +0000 |
---|---|---|
committer | Chris Lamb <lamby@debian.org> | 2016-08-04 17:49:15 +0000 |
commit | a07fab907f1ad57b4f03ebd695f47009a816ce57 (patch) | |
tree | 1e603293200fa7127e7e6071fb7df821f5261e57 /bin/lts-missing-uploads.py | |
parent | 504764c9b2f1dc683c050df8e8f68cf1e0faeb77 (diff) |
lts-missing-uploads: Don't blow up on invalid version numbers in DLAs
git-svn-id: svn+ssh://svn.debian.org/svn/secure-testing@43772 e39458fd-73e7-0310-bf30-c45bca0a0e42
Diffstat (limited to 'bin/lts-missing-uploads.py')
-rwxr-xr-x | bin/lts-missing-uploads.py | 25 |
1 files changed, 19 insertions, 6 deletions
diff --git a/bin/lts-missing-uploads.py b/bin/lts-missing-uploads.py index 462c904e19..0c8cbf9dd0 100755 --- a/bin/lts-missing-uploads.py +++ b/bin/lts-missing-uploads.py @@ -71,16 +71,29 @@ class LTSMissingUploads(object): sources = self.get_sources() for source, dla in sorted(dlas.items()): - version = sources[source] - - if Version(dla['version']) > Version(version): - self.warn("{}: DLA-{} announced version {} but LTS has {} <{}>".format( + try: + dla_version = Version(dla['version']) + except ValueError: + self.warn("{}: DLA-{} announced with invalid version: {} <{}>", source, dla['dla'], dla['version'], - version, dla['url'], - )) + ) + continue + + archive_version = Version(sources[source]) + + if dla_version <= archive_version: + continue + + self.warn("{}: DLA-{} announced version {} but LTS has {} <{}>", + source, + dla['dla'], + dla_version, + archive_version, + dla['url'], + ) return 0 |