Rev 14: Handle old branches that don't support tags. in http://bazaar.launchpad.net/~canonical-bazaar/udd/hottest100

[John Arbash Meinel]

At http://bazaar.launchpad.net/~canonical-bazaar/udd/hottest100

------------------------------------------------------------
revno: 14
revision-id: john at arbash-meinel.com-20100126111215-hf3dixh5p9j25m2k
parent: john at arbash-meinel.com-20100126111040-fjq79yvubyjouarp
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: hottest100
timestamp: Tue 2010-01-26 05:12:15 -0600
message:
  Handle old branches that don't support tags.
  
  Use the soyuz package version to determine if the packaging branch
  is up to date.
  
  Add a repr for Package.
-------------- next part --------------
=== modified file 'check-hottest.py'
--- a/check-hottest.py	2010-01-26 10:46:14 +0000
+++ b/check-hottest.py	2010-01-26 11:12:15 +0000
@@ -43,6 +43,10 @@
         self.name = elements[0]
         self.tags = set(elements[1:])
 
+    def __repr__(self):
+        return '%s(%s, %s)' % (self.__class__.__name__,
+                               self.name, self.tags)
+
 
 def get_packages(name):
     f = open(name, 'rb')
@@ -61,7 +65,11 @@
     try:
         rev = branch.repository.get_revision(branch.last_revision())
         dt = datetime.datetime.fromtimestamp(rev.timestamp)
-        return datetime.datetime.now() - dt, branch.tags.get_tag_dict()
+        try:
+            tag_dict = branch.tags.get_tag_dict()
+        except errors.TagsNotSupported:
+            tag_dict = {}
+        return datetime.datetime.now() - dt, tag_dict
     finally:
         branch.unlock()
 
@@ -148,15 +156,13 @@
             if source_package_version not in tags:
                 missing_soyuz_package.add(package)
                 package_status = 'missing'
+                stale_packages.add(package)
             else:
                 package_status = 'up-to-date'
+                ok_packages.add(package)
             output_stream.write(fmt_str % (
                 package_branch_url, age.days, source_package_version,
                 package_status))
-            if age.days < opts.stale_days:
-                ok_packages.add(package)
-            else:
-                stale_packages.add(package)
         except errors.BzrError, e:
             # the launchpad plugin sends 
             # <class 'bzrlib.errors.InvalidURL'> Invalid url supplied to transport: