[Piuparts-commits] rev 578 - in trunk: . debian

[Holger Levsen]

Author: holger
Date: 2009-12-19 22:09:50 +0000 (Sat, 19 Dec 2009)
New Revision: 578

Modified:
   trunk/debian/changelog
   trunk/piuparts-report.py
Log:
sort some lists alphabetically

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2009-12-18 14:26:41 UTC (rev 577)
+++ trunk/debian/changelog	2009-12-19 22:09:50 UTC (rev 578)
@@ -1,7 +1,8 @@
 piuparts (0.39) UNRELEASED; urgency=low
 
-  * piuparts-report.py: report packages which failed due to broken maintainer
-    scripts.
+  * piuparts-report.py: 
+     - report packages which failed due to broken maintainer scripts.
+     - visual improvements in the output.
 
  -- Holger Levsen <holger at debian.org>  Fri, 18 Dec 2009 13:32:16 +0100
 

Modified: trunk/piuparts-report.py
===================================================================
--- trunk/piuparts-report.py	2009-12-18 14:26:41 UTC (rev 577)
+++ trunk/piuparts-report.py	2009-12-19 22:09:50 UTC (rev 578)
@@ -828,7 +828,7 @@
 
     def create_and_link_to_analysises(self,state):
         link="<ul>"
-        templates = find_files_with_suffix(self._output_directory,".tpl")
+        templates = sorted(find_files_with_suffix(self._output_directory,".tpl"))
         for template in templates:
           if (state == "failed-testing" and template[-9:] != "issue.tpl") or (state == "successfully-tested" and template[-9:] == "issue.tpl"):
 
@@ -886,7 +886,7 @@
         for state in self._binary_db.get_states():
             logging.debug("Writing page for %s" % state)
             list = "<ul>\n"
-            for package in self._binary_db.get_packages_in_state(state):
+            for package in sorted(self._binary_db.get_packages_in_state(state)):
                 list += "<li id=\"%s\">%s (%s)" % (
                                          package["Package"],
                                          self.link_to_source_summary(package["Package"]),
@@ -944,7 +944,7 @@
         self._section_names = section_names
         self._master_directory = os.path.abspath(os.path.join(master_directory, self._config.section))
         if not os.path.exists(self._master_directory):
-            logging.debug("Warning: %s does not exist. Did you ever let the slave work?" % (self._master_directory, self._config.section))
+            logging.debug("Warning: %s did not exist, now created. Did you ever let the slave work?" % (self._master_directory, self._config.section))
             os.mkdir(self._master_directory)
 
         self._output_directory = os.path.abspath(os.path.join(output_directory, self._config.section))