Boost logo

Boost-Build :

From: Jurko Gospodnetiæ (jurko.gospodnetic_at_[hidden])
Date: 2008-07-11 19:10:29


   Hi.

   The attached patch made the capture-output testing rule in python.jam
work correctly when the path to the used Python executable includes spaces.

   It simply quotes the used Python executable before using it in the
actual command line so it should not be destructive. On the other hand,
something like this failing could mess up Boost Build testing which is
something we do not want this close to the release.

   I ran the Boost Python tests with this patch on Windows and all
passed fine.

   Anyone see any objections to commiting this patch?

   Best regards,
     Jurko Gospodnetiæ

Index: tools/build/v2/tools/python.jam
===================================================================
--- tools/build/v2/tools/python.jam (revision 47329)
+++ tools/build/v2/tools/python.jam (working copy)
@@ -1218,7 +1218,7 @@
     local path-separator = [ os.path-separator [ translate-os $(target-os) ] ] ;
     local set-PYTHONPATH = [ common.variable-setting-command PYTHONPATH :
         $(PYTHONPATH:J=$(path-separator)) ] ;
- LAUNCHER on $(target) = $(set-PYTHONPATH) [ on $(target) return $(PYTHON) ] ;
+ LAUNCHER on $(target) = $(set-PYTHONPATH) [ on $(target) return \"$(PYTHON)\" ] ;
 }
 
 


Boost-Build list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk