Fix issue 2628, long compile lines in batch mode. Thanks to Grzegorz Bizo for the patch.

3 files changed, 17 insertions, 12 deletions

diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 97fb063..6c930e6 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -7,6 +7,9 @@
 
 RELEASE 2.1.0.alpha.yyyymmdd - NEW DATE WILL BE INSERTED HERE
 
+  From Grzegorz Bizoń:
+    - Fix long compile lines in batch mode by using TEMPFILE
+
   From Justin Gullingsrud:
     - support -std=c++0x and related CXXFLAGS in pkgconfig (ParseFlags)
 
@@ -80,7 +83,7 @@ RELEASE 2.1.0.alpha.yyyymmdd - NEW DATE WILL BE INSERTED HERE
 
     - Print the path to the SCons package in scons --version
 
-  From Jean-Fran�ois Colson:
+  From Jean-Fran�ois Colson:
 
     - Improve Microsoft Visual Studio Solution generation, and fix
       various errors in the generated solutions especially when using
@@ -642,7 +645,7 @@ RELEASE 1.1.0.d20081207 - Sun, 07 Dec 2008 19:17:23 -0800
 
     - Issue 2401: Fix usage of comparisons with None.
 
-  From Ludwig H�hne:
+  From Ludwig H�hne:
 
     - Handle Java inner classes declared within a method.
 
@@ -771,7 +774,7 @@ RELEASE 1.1.0 - Thu, 09 Oct 2008 08:33:47 -0700
 
     - Fix VariantDir duplication of #included files in subdirectories.
 
-  From Ludwig H�hne:
+  From Ludwig H�hne:
 
     - Reduce memory usage when a directory is used as a dependency of
       another Node (such as an Alias) by returning a concatenation
@@ -886,7 +889,7 @@ RELEASE 1.0.0 - XXX
 
     - Clear the Node state when turning a generic Entry into a Dir.
 
-  From Ludwig H�hne:
+  From Ludwig H�hne:
 
     - Fix sporadic output-order failures in test/GetBuildFailures/parallel.py.
 
@@ -1120,7 +1123,7 @@ RELEASE 0.98.1 - Fri, 18 Apr 2008 19:11:58 -0700
 
   - Fix the --debug=stree option so it prints its tree once, not twice.
 
-  From Johan Boul�:
+  From Johan Boul�:
 
   - Fix the ability to use LoadableModule() under MinGW.
 
@@ -1233,7 +1236,7 @@ RELEASE 0.98 - Sun, 30 Mar 2008 23:33:05 -0700
     calls by using a File's .suffix attribute directly instead of
     stringifying it.
 
-  From Jérôme Berger:
+  From Jérôme Berger:
 
   - Have the D language scanner search for .di files as well as .d files.
 
@@ -2334,7 +2337,7 @@ RELEASE 0.96.92 - Mon, 10 Apr 2006 21:08:22 -0400
   - Fix C/C++ compiler selection on AIX to not always use the external $CC
     environment variable.
 
-  From August Hörandl:
+  From August Hörandl:
 
   - Add a scanner for \include and \import files, with support for
     searching a directory list in $TEXINPUTS (imported from the external
diff --git a/src/RELEASE.txt b/src/RELEASE.txt
index 0832daf..6fde09c 100644
--- a/src/RELEASE.txt
+++ b/src/RELEASE.txt
@@ -56,6 +56,7 @@
 
   FIXES
 
+    - Long compile lines no longer break MSVC_BATCH mode
     - RPATH is now in LINKCOM rather than LINKFLAGS, so resetting
       LINKFLAGS doesn't kill RPATH
     - Precompiled headers on Windows no longer break when used with
@@ -118,7 +119,8 @@
   Thanks to
   Dirk Baechle,
   Vincent Beffara,
-  Jean-François Colson,
+  Grzegorz Bizoń,
+  Jean-François Colson,
   Bauke Conijn,
   Bill Deegan,
   Ken Deeter,
diff --git a/src/engine/SCons/Tool/msvc.py b/src/engine/SCons/Tool/msvc.py
index 066b5e5..36b65ca 100644
--- a/src/engine/SCons/Tool/msvc.py
+++ b/src/engine/SCons/Tool/msvc.py
@@ -214,17 +214,17 @@ def generate(env):
     env['CC']         = 'cl'
     env['CCFLAGS']    = SCons.Util.CLVar('/nologo')
     env['CFLAGS']     = SCons.Util.CLVar('')
-    env['CCCOM']      = '$CC $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $CFLAGS $CCFLAGS $_CCCOMCOM'
+    env['CCCOM']      = '${TEMPFILE("$CC $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $CFLAGS $CCFLAGS $_CCCOMCOM")}'
     env['SHCC']       = '$CC'
     env['SHCCFLAGS']  = SCons.Util.CLVar('$CCFLAGS')
     env['SHCFLAGS']   = SCons.Util.CLVar('$CFLAGS')
-    env['SHCCCOM']    = '$SHCC $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $SHCFLAGS $SHCCFLAGS $_CCCOMCOM'
+    env['SHCCCOM']    = '${TEMPFILE("$SHCC $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $SHCFLAGS $SHCCFLAGS $_CCCOMCOM")}'
     env['CXX']        = '$CC'
     env['CXXFLAGS']   = SCons.Util.CLVar('$( /TP $)')
-    env['CXXCOM']     = '$CXX $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $CXXFLAGS $CCFLAGS $_CCCOMCOM'
+    env['CXXCOM']     = '${TEMPFILE("$CXX $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $CXXFLAGS $CCFLAGS $_CCCOMCOM")}'
     env['SHCXX']      = '$CXX'
     env['SHCXXFLAGS'] = SCons.Util.CLVar('$CXXFLAGS')
-    env['SHCXXCOM']   = '$SHCXX $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $SHCXXFLAGS $SHCCFLAGS $_CCCOMCOM'
+    env['SHCXXCOM']   = '${TEMPFILE("$SHCXX $_MSVC_OUTPUT_FLAG /c $CHANGED_SOURCES $SHCXXFLAGS $SHCCFLAGS $_CCCOMCOM")}'
     env['CPPDEFPREFIX']  = '/D'
     env['CPPDEFSUFFIX']  = ''
     env['INCPREFIX']  = '/I'