From: stevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Date: Sat, 13 Aug 2005 00:56:13 +0000 (+0000)
Subject: Add %SYSTEMDRIVE% to the list of Windows variables preserved.  (Chris Prince)
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=b26c8dfc7cb18eda14aa9432c271bb6cba894ae2;p=scons.git

Add %SYSTEMDRIVE% to the list of Windows variables preserved.  (Chris Prince)


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

diff --git a/doc/man/scons.1 b/doc/man/scons.1
index cb58f68b..5db1f53a 100644
--- a/doc/man/scons.1
+++ b/doc/man/scons.1
@@ -1028,8 +1028,10 @@ and suffixes appropriate for the platform.
 Note that the
 .B win32
 platform adds the
+.B SYSTEMDRIVE
+and
 .B SYSTEMROOT
-variable from the user's external environment
+variables from the user's external environment
 to the construction environment's
 .B ENV
 dictionary.
@@ -3633,8 +3635,10 @@ env.Platform('posix')
 Note that the
 .B win32
 platform adds the
+.B SYSTEMDRIVE
+and
 .B SYSTEMROOT
-variable from the user's external environment
+variables from the user's external environment
 to the construction environment's
 .B ENV
 dictionary.
diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 88b202be..b01a41df 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -448,6 +448,9 @@ RELEASE 0.97 - XXX
   - Look in the right directory, not always the local directory, for a
     same-named file or directory conflict on disk.
 
+  - On Windows, preserve the external environment's %SYSTEMDRIVE%
+    variable, too.
+
   From Kevin Quick:
 
   - Fix the Builder name returned from ListBuilders and other instances
diff --git a/src/engine/SCons/Platform/win32.py b/src/engine/SCons/Platform/win32.py
index 6faf606d..61880987 100644
--- a/src/engine/SCons/Platform/win32.py
+++ b/src/engine/SCons/Platform/win32.py
@@ -232,9 +232,11 @@ def generate(env):
     # environment's ENV.  This is a potential slippery slope, because we
     # *don't* want to make builds dependent on the user's environment by
     # default.  We're doing this for SYSTEMROOT, though, because it's
-    # needed for anything that uses sockets, and seldom changes.  Weigh
-    # the impact carefully before adding other variables to this list.
-    import_env = [ 'SYSTEMROOT', 'TEMP', 'TMP' ]
+    # needed for anything that uses sockets, and seldom changes, and
+    # for SYSTEMDRIVE because it's related.
+    #
+    # Weigh the impact carefully before adding other variables to this list.
+    import_env = [ 'SYSTEMDRIVE', 'SYSTEMROOT', 'TEMP', 'TMP' ]
     for var in import_env:
         v = os.environ.get(var)
         if v: