upgradecheck upgradecheck.py,1.14,1.15

[Michael Schwendt (mschwendt)]

Author: mschwendt

Update of /cvs/fedora/upgradecheck
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv23298

Modified Files:
	upgradecheck.py 
Log Message:
At the top of the report, sort by package owner. At the bottom, sort by package name.


Index: upgradecheck.py
===================================================================
RCS file: /cvs/fedora/upgradecheck/upgradecheck.py,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- upgradecheck.py	1 Aug 2006 19:10:21 -0000	1.14
+++ upgradecheck.py	20 Aug 2006 00:28:08 -0000	1.15
@@ -146,6 +146,7 @@
     allnames.sort(lambda x, y: cmp(x.lower(), y.lower()))
 
     report = []
+    reports = {}  # report per owner, key is owner email addr
 
     for dist in enabled_dists:
         if not known_good.get(dist):
@@ -173,13 +174,35 @@
             ix = ix + 1
         if bork is not None:
             ix = bork
-            report.append(name + ": " + owners[name].replace('@',' AT '))
+            owner = owners[name]
+            ownerprint = owner.replace('@',' AT ')
+            if not reports.has_key(owner):
+                reports[owner] = []
+            reports[owner].append(name)
+            report.append(name+': '+ownerprint)
             for curr in pkgdata[ix:]: # onwards from the problem spot
                 if curr: # we haven't checked if all distros after the bork spot have this package
-                    report.append("  %s: %s (%s)" % (enabled_dists[ix], evrstr(curr["evr"]), curr["repo"]))
+                    what = "  %s: %s (%s)" % (enabled_dists[ix], evrstr(curr["evr"]), curr["repo"])
+                    reports[owner].append(what)
+                    report.append(what)
                 ix = ix + 1
+            reports[owner].append("")
             report.append("")
 
+    # Insert "sorted by owner" report at the top.
+    oldreport = report
+    report = []
+    reportkeys = reports.keys()
+    reportkeys.sort()
+    for owner in reportkeys:
+        ownerprint = owner.replace('@',' AT ')
+        report.append(ownerprint+':')
+        for line in reports[owner]:
+            report.append('    '+line)
+    report.append('-'*70)
+    report.append('')
+    report += oldreport
+        
     report = "\n".join(report)
     if report:
         if mail_to and not opts.nomail: