Fix for leaving EXTRAS blank
authorGabe Black <gblack@eecs.umich.edu>
Mon, 10 Sep 2007 23:42:41 +0000 (16:42 -0700)
committerGabe Black <gblack@eecs.umich.edu>
Mon, 10 Sep 2007 23:42:41 +0000 (16:42 -0700)
Apparently env['EXTRAS'] will return an empty string if not set. split will
then split it into an empty string, and normalize will turn "" into ".".

--HG--
extra : convert_revision : f79efebb129fdd65161fcf4d4582c2a8541aeacd

src/SConscript

index a4bd5526974e0568c9b1bd28c62c4ffb376bcee6..a2df88c0604776e74c24fc345eb669dfe6bbce3d 100644 (file)
@@ -163,15 +163,17 @@ for root, dirs, files in os.walk(srcdir, topdown=True):
         base = root[len(srcdir) + 1:]
         SConscript(joinpath(base, 'SConscript'))
 
-for extra in env['EXTRAS'].split(':'):
-    extra = os.path.expanduser(extra)
-    extra = os.path.normpath(extra)
-    env.Append(CPPPATH=[Dir(extra)])
-    for root, dirs, files in os.walk(extra, topdown=True):
-        if 'SConscript' in files:
-            subdir = root[len(os.path.dirname(extra))+1:]
-            build_dir = joinpath(env['BUILDDIR'], subdir)
-            SConscript(joinpath(root, 'SConscript'), build_dir=build_dir)
+extra_string = env['EXTRAS']
+if extra_string and extra_string != '' and not extra_string.isspace():
+    for extra in extra_string.split(':'):
+        extra = os.path.expanduser(extra)
+        extra = os.path.normpath(extra)
+        env.Append(CPPPATH=[Dir(extra)])
+        for root, dirs, files in os.walk(extra, topdown=True):
+            if 'SConscript' in files:
+                subdir = root[len(os.path.dirname(extra))+1:]
+                build_dir = joinpath(env['BUILDDIR'], subdir)
+                SConscript(joinpath(root, 'SConscript'), build_dir=build_dir)
 
 for opt in env.ExportOptions:
     env.ConfigFile(opt)