Rev 11: Switch to using make_output_stream in http://bazaar.launchpad.net/~canonical-bazaar/udd/hottest100

[John Arbash Meinel]

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

------------------------------------------------------------
revno: 11
revision-id: john at arbash-meinel.com-20100126104614-pp06k60i6x3cx59u
parent: jelmer at samba.org-20100126104359-vzyb3jfh5hmt44ba
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: hottest100
timestamp: Tue 2010-01-26 04:46:14 -0600
message:
  Switch to using make_output_stream
  
  A couple more formatting updates to make failures clearer.
-------------- next part --------------
=== modified file 'check-hottest.py'
--- a/check-hottest.py	2010-01-26 10:40:06 +0000
+++ b/check-hottest.py	2010-01-26 10:46:14 +0000
@@ -111,30 +111,32 @@
 
     today = datetime.datetime.now()
     reuse_transports = []
-    fmt_str = '  %-40s  %-4s %-12s %s'
-    print fmt_str % ('package branch url', 'age', 'soyuz version', 'status')
-    print fmt_str % ('', '(days)', '', '')
+    output_stream = ui.ui_factory.make_output_stream()
+    fmt_str = '  %-40s  %-4s %-12s %s\n'
+    output_stream.write(fmt_str % ('package branch url', 'age', 'soyuz',
+                                   'status'))
+    output_stream.write(fmt_str % ('', '(days)', '', ''))
     for package in packages:
         print package.name
         # handle tags
         if 'old-version' in package.tags:
-            print ' tagged old-version\n'
+            output_stream.write(' tagged old-version\n')
             old_packages.add(package)
             continue
         if 'metapackage' in package.tags:
-            print ' tagged metapackage\n'
+            output_stream.write(' tagged metapackage\n')
             meta_packages.add(package)
             continue
         if 'special' in package.tags:
-            print ' tagged special\n'
+            output_stream.write(' tagged special\n')
             special_packages.add(package)
             continue
         if 'devfocus' in package.tags:
-            print ' tagged devfocus\n'
+            output_stream.write(' tagged devfocus\n')
             devfocus_packages.add(package)
             continue
         if 'hg-import' in package.tags:
-            print ' tagged hg-import\n'
+            output_stream.write(' tagged hg-import\n')
             hg_packages.add(package)
             continue
         try:
@@ -148,9 +150,9 @@
                 package_status = 'missing'
             else:
                 package_status = 'up-to-date'
-            print fmt_str % (
+            output_stream.write(fmt_str % (
                 package_branch_url, age.days, source_package_version,
-                package_status)
+                package_status))
             if age.days < opts.stale_days:
                 ok_packages.add(package)
             else:
@@ -160,26 +162,32 @@
             # <class 'bzrlib.errors.InvalidURL'> Invalid url supplied to transport:
             # "lp:ubuntu/hal": hal in ubuntu has no default branch.
             if str(e).find('in ubuntu has no default branch') > -1:
-                print '  %s has no default branch' % (package_branch_url,)
+                output_stream.write('  %s has no default branch\n'
+                                    % (package_branch_url,))
                 unregistered_packages.add(package)
             else:
-                print '  %s' % (e,)
+                output_stream.write('  %s\n' % (e,))
                 broken_packages.add(package)
         try:
             upstream_branch_url = 'lp:%s' % (package.name,)
             age, _ = get_branch_age_and_tags(upstream_branch_url)
-            print fmt_str % (upstream_branch_url, age.days, '', '')
+            output_stream.write(fmt_str
+                                % (upstream_branch_url, age.days, '', ''))
             if age.days < opts.stale_days:
                 ok_upstreams.add(package)
             else:
                 stale_upstreams.add(package)
         except errors.BzrError, e:
-            print '  %s' % (e,)
             if "No such project" in str(e):
+                output_stream.write('  %s has no project\n'
+                                    % (upstream_branch_url,))
                 missing_projects.add(package)
             elif 'has no default branch' in str(e):
+                output_stream.write('  %s has no default branch\n'
+                                    % (upstream_branch_url,))
                 no_default_upstream_branch.add(package)
             else:
+                output_stream.write('  %s\n' % (e,))
                 broken_upstreams.add(package)
 
     print 'TOTALS:'