# HG changeset patch
# User Bernhard Herzog <bh@intevation.de>
# Date 1226691663 0
# Node ID cbcd8f549ff7a8f82d5b0d3dc5936175f4b7b440
# Parent  1527c37bd7aa1919abba32bfbd83e4aeaad1383f
Make logfile and binary_dir parameters of PBuilder.build optional.
Adapt test cases to changed order of pbuild command line arguments.

diff -r 1527c37bd7aa -r cbcd8f549ff7 test/test_builder.py
--- a/test/test_builder.py	Thu Nov 06 15:59:50 2008 +0000
+++ b/test/test_builder.py	Fri Nov 14 19:41:03 2008 +0000
@@ -190,10 +190,10 @@
                                   "/home/builder/pbuilder"])
         self.check_command_line(['/usr/sbin/pbuilder', 'build',
                                  '--configfile', 'my_pbuilderrc',
+                                 '--logfile', logfilename,
+                                 '--buildresult', binary_dir_name,
                                  '--bindmounts', "/home/builder/tracks",
                                  '--bindmounts', "/home/builder/pbuilder",
-                                 '--logfile', logfilename,
-                                 '--buildresult', binary_dir_name,
                                  '--debbuildopts', '-b',
                                  'my_dsc_file'])
         self.failUnless(os.path.isdir(binary_dir_name))
@@ -213,10 +213,10 @@
                       extra_packages=["subversion", "texinfo"])
         self.check_command_line(['/usr/sbin/pbuilder', 'build',
                                  '--configfile', 'my_pbuilderrc',
+                                 '--logfile', logfilename,
+                                 '--buildresult', binary_dir_name,
                                  '--extrapackages', "subversion",
                                  '--extrapackages', "texinfo",
-                                 '--logfile', logfilename,
-                                 '--buildresult', binary_dir_name,
                                  '--debbuildopts', '-b',
                                  'my_dsc_file'])
         self.failUnless(os.path.isdir(binary_dir_name))
diff -r 1527c37bd7aa -r cbcd8f549ff7 treepkg/builder.py
--- a/treepkg/builder.py	Thu Nov 06 15:59:50 2008 +0000
+++ b/treepkg/builder.py	Fri Nov 14 19:41:03 2008 +0000
@@ -152,7 +152,7 @@
         finally:
             script.close()
 
-    def build(self, dsc_file, binary_dir, logfile, bindmounts=(),
+    def build(self, dsc_file, binary_dir=None, logfile=None, bindmounts=(),
               extra_packages=(), extra_env=None):
         """Build a binary packager from a source package
         Parameters:
@@ -169,26 +169,29 @@
                         not remove these variables when it starts
                         pbuilder
         """
-        util.ensure_directory(binary_dir)
         args = []
+        if logfile is not None:
+            args.extend(["--logfile", logfile])
+        if binary_dir is not None:
+            args.extend(["--buildresult", binary_dir])
+            util.ensure_directory(binary_dir)
         for mount in bindmounts:
             args.extend(["--bindmounts", mount])
         for pkg in extra_packages:
             args.extend(["--extrapackages", pkg])
         run.call(cmdexpand("@rootcmd /usr/sbin/pbuilder build"
                            " --configfile $pbuilderrc @args"
-                           " --logfile $logfile --buildresult $bindir"
                            " --debbuildopts -b $dsc",
                            rootcmd=self.root_cmd, pbuilderrc=self.pbuilderrc,
-                           logfile=logfile, bindir=binary_dir, dsc=dsc_file,
-                           args=args),
+                           dsc=dsc_file, args=args),
                  suppress_output=True,
                  extra_env=extra_env)
         # remove the source package files put into the binary directory
         # by pbuilder
-        for filename in os.listdir(binary_dir):
-            if os.path.splitext(filename)[1] not in (".deb", ".changes"):
-                os.remove(os.path.join(binary_dir, filename))
+        if binary_dir is not None:
+            for filename in os.listdir(binary_dir):
+                if os.path.splitext(filename)[1] not in (".deb", ".changes"):
+                    os.remove(os.path.join(binary_dir, filename))
 
     def add_binaries_to_extra_pkg(self, filenames):
         """Adds binary packages to the extra-pkg directory.