From 93acc2d810798fda05fcb463da6362baa8653fd7 Mon Sep 17 00:00:00 2001 From: Steven Knight Date: Wed, 16 Feb 2005 22:34:26 +0000 Subject: Fix creating a build_dir from scratch when there's a subsidiary SConscript() file. --- src/engine/SCons/Node/FS.py | 58 +++++++++++++----------- src/engine/SCons/Script/SConscript.py | 4 +- test/SConscript-build_dir.py | 85 ++++++++++++++++++++++------------- 3 files changed, 89 insertions(+), 58 deletions(-) diff --git a/src/engine/SCons/Node/FS.py b/src/engine/SCons/Node/FS.py index 2830a20..e7b283d 100644 --- a/src/engine/SCons/Node/FS.py +++ b/src/engine/SCons/Node/FS.py @@ -1286,6 +1286,36 @@ class Dir(Base): if not self.builder is MkdirBuilder: apply(SCons.Node.Node.build, [self,], kw) + def _create(self): + """Create this directory, silently and without worrying about + whether the builder is the default or not.""" + listDirs = [] + parent = self + while parent: + if parent.exists(): + break + listDirs.append(parent) + p = parent.up() + if isinstance(p, ParentOfRoot): + raise SCons.Errors.StopError, parent.path + parent = p + listDirs.reverse() + for dirnode in listDirs: + try: + # Don't call dirnode.build(), call the base Node method + # directly because we definitely *must* create this + # directory. The dirnode.build() method will suppress + # the build if it's the default builder. + SCons.Node.Node.build(dirnode) + dirnode.get_executor().nullify() + # The build() action may or may not have actually + # created the directory, depending on whether the -n + # option was used or not. Delete the _exists and + # _rexists attributes so they can be reevaluated. + dirnode.clear() + except OSError: + pass + def multiple_side_effect_has_builder(self): global MkdirBuilder return not self.builder is MkdirBuilder and self.has_builder() @@ -1542,33 +1572,7 @@ class File(Base): def _createDir(self): # ensure that the directories for this node are # created. - - listDirs = [] - parent=self.dir - while parent: - if parent.exists(): - break - listDirs.append(parent) - p = parent.up() - if isinstance(p, ParentOfRoot): - raise SCons.Errors.StopError, parent.path - parent = p - listDirs.reverse() - for dirnode in listDirs: - try: - # Don't call dirnode.build(), call the base Node method - # directly because we definitely *must* create this - # directory. The dirnode.build() method will suppress - # the build if it's the default builder. - SCons.Node.Node.build(dirnode) - dirnode.get_executor().nullify() - # The build() action may or may not have actually - # created the directory, depending on whether the -n - # option was used or not. Delete the _exists and - # _rexists attributes so they can be reevaluated. - dirnode.clear() - except OSError: - pass + self.dir._create() def retrieve_from_cache(self): """Try to retrieve the node's content from a cache diff --git a/src/engine/SCons/Script/SConscript.py b/src/engine/SCons/Script/SConscript.py index 2e8c916..d0df6a3 100644 --- a/src/engine/SCons/Script/SConscript.py +++ b/src/engine/SCons/Script/SConscript.py @@ -230,7 +230,9 @@ def _SConscript(fs, *files, **kw): # Repository directory. Like above, we do this # directly. fs.chdir(frame.prev_dir, change_os_dir=0) - os.chdir(frame.prev_dir.rdir().get_abspath()) + rdir = frame.prev_dir.rdir() + rdir._create() # Make sure there's a directory there. + os.chdir(rdir.get_abspath()) results.append(frame.retval) diff --git a/test/SConscript-build_dir.py b/test/SConscript-build_dir.py index 91352e1..298eb9b 100644 --- a/test/SConscript-build_dir.py +++ b/test/SConscript-build_dir.py @@ -44,7 +44,7 @@ all9 = test.workpath('test', 'build', 'var9', 'src', 'all') test.subdir('test') -test.write('test/SConstruct', """ +test.write(['test', 'SConstruct'], """ src = Dir('src') alt = Dir('alt') var1 = Dir('build/var1') @@ -120,19 +120,19 @@ test.run(chdir='test', arguments = '. ../build') all_src = "test/src/aaa.in\ntest/src/bbb.in\ntest/src/ccc.in\n" all_alt = "test/alt/aaa.in\ntest/alt/bbb.in\ntest/alt/ccc.in\n" -test.fail_test(test.read(all1) != all_src) -test.fail_test(test.read(all2) != all_src) -test.fail_test(test.read(all3) != all_src) +test.must_match(all1, all_src) +test.must_match(all2, all_src) +test.must_match(all3, all_src) #XXX We can't support var4 and var5 yet, because our BuildDir linkage #XXX is to an entire source directory. We haven't yet generalized our #XXX infrastructure to be able to take the SConscript file from one source #XXX directory, but the rest of the files from a different one. -#XXX test.fail_test(test.read(all4) != all_alt) -#XXX test.fail_test(test.read(all5) != all_alt) -test.fail_test(test.read(all6) != all_src) -test.fail_test(test.read(all7) != all_src) -test.fail_test(test.read(all8) != all_src) -test.fail_test(test.read(all9) != all_src) +#XXX test.must_match(all4, all_alt) +#XXX test.must_match(all5, all_alt) +test.must_match(all6, all_src) +test.must_match(all7, all_src) +test.must_match(all8, all_src) +test.must_match(all9, all_src) import os import stat @@ -145,30 +145,30 @@ def equal_stats(x,y): # Make sure we did duplicate the source files in build/var1, # and that their stats are the same: for file in ['aaa.in', 'bbb.in', 'ccc.in']: - test.fail_test(not os.path.exists(test.workpath('test', 'build', 'var1', file))) + test.must_exist(test.workpath('test', 'build', 'var1', file)) test.fail_test(not equal_stats(test.workpath('test', 'build', 'var1', file), test.workpath('test', 'src', file))) # Make sure we did duplicate the source files in build/var2, # and that their stats are the same: for file in ['aaa.in', 'bbb.in', 'ccc.in']: - test.fail_test(not os.path.exists(test.workpath('test', 'build', 'var2', file))) + test.must_exist(test.workpath('test', 'build', 'var2', file)) test.fail_test(not equal_stats(test.workpath('test', 'build', 'var2', file), test.workpath('test', 'src', file))) # Make sure we didn't duplicate the source files in build/var3. -test.fail_test(os.path.exists(test.workpath('test', 'build', 'var3', 'aaa.in'))) -test.fail_test(os.path.exists(test.workpath('test', 'build', 'var3', 'bbb.in'))) -test.fail_test(os.path.exists(test.workpath('test', 'build', 'var3', 'ccc.in'))) +test.must_not_exist(test.workpath('test', 'build', 'var3', 'aaa.in')) +test.must_not_exist(test.workpath('test', 'build', 'var3', 'bbb.in')) +test.must_not_exist(test.workpath('test', 'build', 'var3', 'ccc.in')) #XXX We can't support var4 and var5 yet, because our BuildDir linkage #XXX is to an entire source directory. We haven't yet generalized our #XXX infrastructure to be able to take the SConscript file from one source #XXX directory, but the rest of the files from a different one. #XXX Make sure we didn't duplicate the source files in build/var4. -#XXXtest.fail_test(os.path.exists(test.workpath('test', 'build', 'var4', 'aaa.in'))) -#XXXtest.fail_test(os.path.exists(test.workpath('test', 'build', 'var4', 'bbb.in'))) -#XXXtest.fail_test(os.path.exists(test.workpath('test', 'build', 'var4', 'ccc.in'))) +#XXXtest.must_not_exist(test.workpath('test', 'build', 'var4', 'aaa.in')) +#XXXtest.must_not_exist(test.workpath('test', 'build', 'var4', 'bbb.in')) +#XXXtest.must_not_exist(test.workpath('test', 'build', 'var4', 'ccc.in')) #XXX We can't support var4 and var5 yet, because our BuildDir linkage #XXX is to an entire source directory. We haven't yet generalized our @@ -177,26 +177,26 @@ test.fail_test(os.path.exists(test.workpath('test', 'build', 'var3', 'ccc.in'))) #XXX Make sure we did duplicate the source files in build/var5, #XXX and that their stats are the same: #XXXfor file in ['aaa.in', 'bbb.in', 'ccc.in']: -#XXX test.fail_test(not os.path.exists(test.workpath('build', 'var5', file))) +#XXX test.must_exist(test.workpath('build', 'var5', file)) #XXX test.fail_test(not equal_stats(test.workpath('build', 'var5', file), #XXX test.workpath('test', 'src', file))) # Make sure we did duplicate the source files in build/var6, # and that their stats are the same: for file in ['aaa.in', 'bbb.in', 'ccc.in']: - test.fail_test(not os.path.exists(test.workpath('build', 'var6', file))) + test.must_exist(test.workpath('build', 'var6', file)) test.fail_test(not equal_stats(test.workpath('build', 'var6', file), test.workpath('test', 'src', file))) # Make sure we didn't duplicate the source files in build/var7. -test.fail_test(os.path.exists(test.workpath('build', 'var7', 'aaa.in'))) -test.fail_test(os.path.exists(test.workpath('build', 'var7', 'bbb.in'))) -test.fail_test(os.path.exists(test.workpath('build', 'var7', 'ccc.in'))) +test.must_not_exist(test.workpath('build', 'var7', 'aaa.in')) +test.must_not_exist(test.workpath('build', 'var7', 'bbb.in')) +test.must_not_exist(test.workpath('build', 'var7', 'ccc.in')) # Make sure we didn't duplicate the source files in build/var8. -test.fail_test(os.path.exists(test.workpath('build', 'var8', 'aaa.in'))) -test.fail_test(os.path.exists(test.workpath('build', 'var8', 'bbb.in'))) -test.fail_test(os.path.exists(test.workpath('build', 'var8', 'ccc.in'))) +test.must_not_exist(test.workpath('build', 'var8', 'aaa.in')) +test.must_not_exist(test.workpath('build', 'var8', 'bbb.in')) +test.must_not_exist(test.workpath('build', 'var8', 'ccc.in')) ################### test.subdir('test2') @@ -230,9 +230,34 @@ test.run(chdir="test2") _obj = TestSCons._obj -test.fail_test(os.path.exists(test.workpath('test2', 'foo' + _obj))) -test.fail_test(os.path.exists(test.workpath('test2', 'bar' + _obj))) -test.fail_test(not os.path.exists(test.workpath('test2', 'Build', 'foo' + _obj))) -test.fail_test(not os.path.exists(test.workpath('test2', 'Build', 'bar' + _obj))) +test.must_not_exist(test.workpath('test2', 'foo' + _obj)) +test.must_not_exist(test.workpath('test2', 'bar' + _obj)) +test.must_exist(test.workpath('test2', 'Build', 'foo' + _obj)) +test.must_exist(test.workpath('test2', 'Build', 'bar' + _obj)) + +################### +# Make sure that directories for subsidiary SConscript() calls +# in a build_dir get created if they don't already exist. +test.subdir('test3') + +test.subdir(['test3', 'src'], ['test3', 'src', '_glscry']) + +test.write(['test3', 'SConstruct'], """\ +SConscript(dirs=['src'], build_dir='build', duplicate=0) +""") + +test.write(['test3', 'src', 'SConscript'], """\ +SConscript(dirs=['_glscry']) +""") + +test.write(['test3', 'src', '_glscry', 'SConscript'], """\ +""") + +test.write(['test3', 'src', 'file.in'], "file.in\n") + +test.write(['test3', 'src', '_glscry', 'file.in'], "file.in\n") + +test.run(chdir='test3') + test.pass_test() -- cgit v0.12