Merge pull request #6 from bdbaddog/fix_to_String_for_subst

Fix issue where code in utility routine to_String_for_subst() had code where result was dropped

3 files changed, 11 insertions, 4 deletions

diff --git a/.gitignore b/.gitignore
index a5ff571..67d4e35 100644
--- a/.gitignore
+++ b/.gitignore
@@ -35,3 +35,8 @@ venv.bak/
 
 # SCons files
 .sconsign.*
+
+# Tool output
+.coverage
+htmlcov
+
diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 3b717ff..a7ad6f2 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -11,6 +11,11 @@ RELEASE  VERSION/DATE TO BE FILLED IN LATER
 
     - Whatever John Doe did.
 
+  From William Deegan:
+    - Fix issue where code in utility routine to_String_for_subst() had code whose result was never
+      properly returned.  
+      (Found by: James Rinkevich https://pairlist4.pair.net/pipermail/scons-users/2017-October/006358.html )
+
   From Thomas Berg:
     - Fixed a regression in scons-3.0.0 where "from __future__ import print_function" was imposed
       on the scope where SConstruct is executed, breaking existing builds using PY 2.7.
diff --git a/src/engine/SCons/Util.py b/src/engine/SCons/Util.py
index f478399..11bcf2e 100644
--- a/src/engine/SCons/Util.py
+++ b/src/engine/SCons/Util.py
@@ -481,10 +481,7 @@ def to_String_for_subst(s,
     if isinstance(s, BaseStringTypes):
         return s
     elif isinstance(s, SequenceTypes):
-        l = []
-        for e in s:
-            l.append(to_String_for_subst(e))
-        return ' '.join( s )
+        return ' '.join([to_String_for_subst(e) for e in s])
     elif isinstance(s, UserString):
         # s.data can only be either a unicode or a regular
         # string. Please see the UserString initializer.