Don't update the .sconsign files when using -n. (Steve Leblanc)

3 files changed, 22 insertions, 2 deletions

diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 7c1f03c..aa18bf6 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -18,6 +18,17 @@ RELEASE 0.15 - XXX
   - Fix man page bugs: remove duplicate AddPostAction() description;
     document no_import_lib.
 
+  - Eliminate a dependency on the distutils.fancy_getopt module by
+    copying and pasting its wrap_text() function directly.
+
+  From Steve Leblanc:
+
+  - Don't update the .sconsign files when run with -n.
+
+  From David Snopek and Christoph Wiedemann
+
+  - Fix use of the SConf subsystem with SConscriptChdir().
+
 
 
 RELEASE 0.14 - Wed, 21 May 2003 05:16:32 -0500
diff --git a/src/engine/SCons/Script/__init__.py b/src/engine/SCons/Script/__init__.py
index c5e469a..1c773f0 100644
--- a/src/engine/SCons/Script/__init__.py
+++ b/src/engine/SCons/Script/__init__.py
@@ -959,7 +959,8 @@ def _main():
         jobs.run()
     finally:
         display("scons: " + closing_message)
-        SCons.Sig.write()
+        if not options.noexec:
+            SCons.Sig.write()
 
 def main():
     global exit_status
diff --git a/test/option-n.py b/test/option-n.py
index b71648d..bccb785 100644
--- a/test/option-n.py
+++ b/test/option-n.py
@@ -109,7 +109,15 @@ test.fail_test(os.path.exists(test.workpath('f2.out')))
 
 test.run(arguments = args)
 test.fail_test(not os.path.exists(test.workpath('f1.out')))
-test.fail_test(not os.path.exists(test.workpath('f2.out')))
+
+# Test that SCons does not write a modified .sconsign when -n is used.
+expect = test.wrap_stdout("""\
+%s build.py f1.out
+""" % python)
+test.unlink('.sconsign')
+test.write('f1.out', "X1.out\n")
+test.run(arguments = '-n f1.out', stdout = expect)
+test.run(arguments = '-n f1.out', stdout = expect)
 
 def wrap_clean_stdout(string):
     return "scons: Reading SConscript files ...\n" + \