Out of source build support

Author: Justin Nolan

SConstruct

@@ -8,6 +8,10 @@ from typing import List, Union
 
 import SCons
 
+_SCRIPTS_DIR = Dir(".").abspath
+if _SCRIPTS_DIR not in sys.path:
+    sys.path.insert(0, _SCRIPTS_DIR)
+
 # Local
 from build.option_handler import OptionsClass
 from build.glob_recursive import GlobRecursive
@@ -167,9 +171,14 @@ def SetupOptions():
         )
     )
 
-    # Add platform options
+    # Add platform options. Iterate deterministically with the current platform
+    # registered last so its defaults win 
     tools = {}
-    for pl in set(platforms) - set(unsupported_known_platforms):
+    current_platform = env.get("platform", default_platform)
+    supported = sorted(set(platforms) - set(unsupported_known_platforms))
+    if current_platform in supported:
+        supported = [pl for pl in supported if pl != current_platform] + [current_platform]
+    for pl in supported:
         tool = Tool(pl, toolpath=env.TOOLPATH)
         if hasattr(tool, "options"):
             tool.options(opts)
@@ -261,10 +270,11 @@ def FinalizeOptions():
     target_tool.generate(env)
     tool.generate(env)
 
+    Decider("MD5-timestamp")
+
     scons_cache_path = os.environ.get("SCONS_CACHE")
     if scons_cache_path != None:
         CacheDir(scons_cache_path)
-        Decider("MD5")
         print("Scons cache enabled... (path: '" + scons_cache_path + "')")
 
     if env["compiledb"] and is_standalone:

build/__init__.py

@@ -1,5 +1,6 @@
 def GlobRecursive(pattern, nodes=["."], exclude=None):
     import fnmatch
+    import os
 
     import SCons
 
@@ -11,15 +12,17 @@ def GlobRecursive(pattern, nodes=["."], exclude=None):
 
     results = []
     for node in nodes:
-        nnodes = []
-        for f in Glob(str(node) + "/*", source=True):
+        node_str = str(node)
+
+        for f in Glob(node_str + "/*", source=True):
             if type(f) is SCons.Node.FS.Dir:
-                nnodes.append(f)
-        results += GlobRecursive(pattern, nnodes)
-        results += Glob(str(node) + "/" + pattern, source=True)
-        if isinstance(exclude, list):
-            for val in results:
-                for pat in exclude:
-                    if fnmatch.fnmatch(str(val), pat):
-                        results.remove(val)
+                child = node_str + "/" + os.path.basename(str(f))
+                results += GlobRecursive(pattern, [child])
+
+        results += Glob(node_str + "/" + pattern)
+
+    if isinstance(exclude, list):
+        norm = lambda s: str(s).replace("\\", "/")
+        norm_exclude = [norm(p) for p in exclude]
+        results = [r for r in results if not any(fnmatch.fnmatch(norm(r), p) for p in norm_exclude)]
     return results