Fix handling when BuildDir exists but is unwriteable. Add Stop. to messages at SCons...

git-svn-id: http://scons.tigris.org/svn/scons/trunk@1089 fdb21ef1-2011-0410-befe-b5e4ea1792b1

diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 3fc014f3fce5dd192c5cd3ada0e9c83a64e35f2a..000c9438e3c74bf1f2c6d5a2fd7dadab64ba38a6 100644 (file)
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -123,6 +123,9 @@ RELEASE 0.97 - XXX
     function, use its strfunction() method (if any) to display the
     action.
 
+  - Fix handling when BuildDir() exists but is unwriteable.  Add
+    "Stop." to those error messages for consistency.
+
   From Christoph Wiedemann:
 
   - Add an Environment.SetDefault() method that only sets values if

diff --git a/src/engine/SCons/Node/FS.py b/src/engine/SCons/Node/FS.py
index 7f60c7b66143513458a2e4375ea8dd9c8f5faf99..5015c8423895ab11e5347cf27195775a21c65ca7 100644 (file)
--- a/src/engine/SCons/Node/FS.py
+++ b/src/engine/SCons/Node/FS.py
@@ -1675,7 +1675,7 @@ class File(Base):
                 try:
                     Link(self, src, None)
                 except SCons.Errors.BuildError, e:
-                    desc = "Cannot duplicate `%s' in `%s': %s." % (src, self.dir, e.errstr)
+                    desc = "Cannot duplicate `%s' in `%s': %s." % (src.path, self.dir.path, e.errstr)
                     raise SCons.Errors.StopError, desc
                 self.linked = 1
                 # The Link() action may or may not have actually

diff --git a/src/engine/SCons/Script/__init__.py b/src/engine/SCons/Script/__init__.py
index 160fae30f70378dcb44457e98b31252e28ce73d3..4832b020cfac867f860a6b3e82d5ebc9bc4a44f1 100644 (file)
--- a/src/engine/SCons/Script/__init__.py
+++ b/src/engine/SCons/Script/__init__.py
@@ -912,9 +912,9 @@ def _main(args, parser):
         # couldn't be copied in to the BuildDir.  Since we're just
         # reading SConscript files and haven't started building
         # things yet, stop regardless of whether they used -i or -k
-        # or anything else, but don't say "Stop." on the message.
+        # or anything else.
         global exit_status
-        sys.stderr.write("scons: *** %s\n" % e)
+        sys.stderr.write("scons: *** %s  Stop.\n" % e)
         exit_status = 2
         sys.exit(exit_status)
     global sconscript_time

diff --git a/test/BuildDir-errors.py b/test/BuildDir-errors.py
index 967384cd369ad37a3f726c68efb0e793878b6731..93cd3ecedec313be9984b2da0b93f54aa241afb9 100644 (file)
--- a/test/BuildDir-errors.py
+++ b/test/BuildDir-errors.py
@@ -66,6 +66,12 @@ def cat(env, source, target):
 
 env = Environment(BUILDERS={'Build':Builder(action=cat)},
                   SCANNERS=[Scanner(fake_scan, skeys = ['.in'])])
+
+# Do some Node test operations to ensure no side-effects cause failures
+File('file.in').exists()
+File('file.in').is_derived()
+File('file.in').is_pseudo_derived()
+
 env.Build('file.out', 'file.in')
 """)
 
@@ -87,7 +93,7 @@ if sys.platform != 'win32':
     test.run(chdir = 'ro-dir',
              arguments = ".",
              status = 2,
-             stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.\n" % os.path.join('src', 'SConscript'))
+             stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.  Stop.\n" % os.path.join('src', 'SConscript'))
 
 # Verify the error when the SConscript file within the BuildDir is
 # read-only.  Note that we have to make the directory read-only too,
@@ -104,7 +110,7 @@ os.chmod(dir, os.stat(dir)[stat.ST_MODE] & ~stat.S_IWUSR)
 test.run(chdir = 'ro-SConscript',
          arguments = ".",
          status = 2,
-         stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.\n" % os.path.join('src', 'SConscript'))
+         stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.  Stop.\n" % os.path.join('src', 'SConscript'))
 
 os.chmod('ro-SConscript', os.stat('ro-SConscript')[stat.ST_MODE] | stat.S_IWUSR)
 f.close()
@@ -112,7 +118,7 @@ f.close()
 test.run(chdir = 'ro-SConscript',
          arguments = ".",
          status = 2,
-         stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.\n" % os.path.join('src', 'SConscript'))
+         stderr = "scons: *** Cannot duplicate `%s' in `build': Permission denied.  Stop.\n" % os.path.join('src', 'SConscript'))
 
 # Verify the error when the source file within the BuildDir is
 # read-only.  Note that we have to make the directory read-only too,
@@ -140,7 +146,7 @@ test.run(chdir = 'ro-src',
          arguments = "-k .",
          status = 2,
          stderr = """\
-scons: *** Cannot duplicate `%s' in `build': Permission denied.
+scons: *** Cannot duplicate `%s' in `build': Permission denied.  Stop.
 """ % (os.path.join('src', 'file.in')))
 
 f.close()