[Piuparts-commits] [SCM] piuparts git repository branch, develop, updated. 28129dbac5ba28660ad06a24250cdcc6204d354d

[Holger Levsen]

The following commit has been merged in the develop branch:
commit 28129dbac5ba28660ad06a24250cdcc6204d354d
Author: Holger Levsen <holger at layer-acht.org>
Date:   Sun Dec 11 16:51:46 2011 +0100

    improve last commit

diff --git a/debian/changelog b/debian/changelog
index c053521..b04b4ed 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,7 +8,7 @@ piuparts (0.42) UNRELEASED; urgency=low
   * Remove Debian.NEWS entry about source in git. (Closes: #640121)
   * piuparts.py, piuparts-report.py, ChangeLog: Expand tabs to spaces.
   * Remove whitespaces from whitespace-only lines.
-  * piuparts-report: create maintainer subdir if it doesn't exist.
+  * piuparts-report: create maintainer subdirs if they don't exist.
 
   [ Mika Pflüger ]
   * piuparts-analyze.py:
diff --git a/piuparts-report.py b/piuparts-report.py
index 6715702..ea368d6 100644
--- a/piuparts-report.py
+++ b/piuparts-report.py
@@ -729,6 +729,9 @@ class Section:
 
     def create_maintainer_summaries(self, maintainers, source_data):
         logging.debug("Writing maintainer summaries in %s" % self._output_directory)
+        maintainer_dir = os.path.join(self._output_directory, "maintainer")
+        if not os.path.exists(maintainer_dir):
+            os.mkdir(maintainer_dir)
         states = ["fail", "unknown", "pass"]
         for maintainer in maintainers.keys():
             sources = maintainers[maintainer]
@@ -759,10 +762,10 @@ class Section:
             distrolinks += "</td></tr>"
 
             htmlpage = string.Template(HTML_HEADER + MAINTAINER_BODY_TEMPLATE + HTML_FOOTER)
-            maintainer_path = os.path.join(self._output_directory, "maintainer", maintainer_subdir(maintainer))
-            if not os.path.exists(maintainer_path):
-              os.mkdir(maintainer_path)
-            filename = os.path.join(maintainer_path, maintainer + ".html")
+            maintainer_subdir = os.path.join(maintainer_dir, maintainer_subdir(maintainer))
+            if not os.path.exists(maintainer_subdir):
+              os.mkdir(maintainer_subdir)
+            filename = os.path.join(maintainer_subdir, maintainer + ".html")
             f = file(filename, "w")
             f.write(htmlpage.safe_substitute( {
                "page_title": html_protect("Status of "+maintainer+" packages in "+self._config.section),

-- 
piuparts git repository