And erm, assuming it is acceptable...

On Wed, Sep 19, 2007 at 08:00:16PM -0400, Phil Frost wrote:
> I see there is an option in SConscript to enable scons' build cache; but
> it a boolean and has a hard-coded path of ./obj. It would be more useful
> for me if it was somewhere outside of the working tree so that different
> branches and different developers can share it. Would anyone be opposed
> to making this a string option which specifies the path to the cache, or
> disables the cache if it is an empty string?
=== modified file 'SConscript'
--- SConscript  2007-09-20 00:29:41 +0000
+++ SConscript  2007-09-19 23:39:38 +0000
@@ -52,7 +52,7 @@
     BoolOption('release', 'Build release version', True),
     BoolOption('simd', 'Build with simd instructions', True),
     BoolOption('distdoxygen', 'Add doxygen docs to tarball', False),
-    PathOption('cache', 'Path to scons\'s compile cache (empty string to 
disable)', None, lambda key, val, env: val == '' or 
PathOption.PathIsDirCreate(key, val, env)),
+    BoolOption('cache', 'Use scons\'s compile cache', False),
     BoolOption('compact_build', 'combine source files in build.cpp files 
(don\'t do this, if you have less than 1gb of ram)', True),
     PathOption('extra_path', 'extra search path', None)
     )
@@ -266,8 +266,7 @@
 #
 
 if env['cache']:
-    print 'using cache at %s' % (env['cache'])
-    env.CacheDir(env['cache'])
+    env.CacheDir("obj")
 
 now = datetime.datetime.now()
 

_______________________________________________
nova-dev mailing list
[email protected]
http://klingt.org/cgi-bin/mailman/listinfo/nova-dev
http://tim.klingt.org/nova

Reply via email to