From: Fred Drake <fdrake@acm.org>
Date: Tue, 17 Jul 2001 14:46:09 +0000 (+0000)
Subject: Job.__init__():  Only create the builddir directory if it will be used (by
X-Git-Tag: v2.2a3~1112
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=aebbca3b610eccde876b243c54d75cae250c36f7;p=thirdparty%2FPython%2Fcpython.git

Job.__init__():  Only create the builddir directory if it will be used (by
    building the HTML or text documentation).  There is no need to create
    it for other formats.

Job.build_html():  The builddir parameter is always passed in, so it need
    not be optional.
---

diff --git a/Doc/tools/mkhowto b/Doc/tools/mkhowto
index 5b69fe3c712a..0d5d687e2981 100755
--- a/Doc/tools/mkhowto
+++ b/Doc/tools/mkhowto
@@ -227,9 +227,12 @@ class Job:
         self.doctype = get_doctype(path)
         self.filedir, self.doc = split_pathname(path)
         self.builddir = os.path.abspath(options.builddir or self.doc)
-        if not os.path.exists(self.builddir):
-            os.mkdir(self.builddir)
-        self.log_filename = os.path.join(self.builddir, self.doc + ".how")
+        if ("html" in options.formats or "text" in options.formats):
+            if not os.path.exists(self.builddir):
+                os.mkdir(self.builddir)
+            self.log_filename = os.path.join(self.builddir, self.doc + ".how")
+        else:
+            self.log_filename = os.path.abspath(self.doc + ".how")
         if os.path.exists(self.log_filename):
             os.unlink(self.log_filename)
         if os.path.exists(self.doc + ".l2h"):
@@ -350,9 +353,7 @@ class Job:
     def build_ps(self):
         self.run("%s -N0 -o %s.ps %s" % (DVIPS_BINARY, self.doc, self.doc))
 
-    def build_html(self, builddir=None, max_split_depth=None):
-        if builddir is None:
-            builddir = self.builddir
+    def build_html(self, builddir, max_split_depth=None):
         if max_split_depth is None:
             max_split_depth = self.options.max_split_depth
         texfile = None