]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[1545] Corrected problems in setting environment variable in tests
authorStephen Morris <stephen@isc.org>
Fri, 14 Sep 2012 14:16:56 +0000 (15:16 +0100)
committerStephen Morris <stephen@isc.org>
Fri, 14 Sep 2012 14:16:56 +0000 (15:16 +0100)
For some reason, the Python "os.putenv()" did not work, so changed
to morifying the "os.environ" variable directly.

src/bin/dhcp4/tests/dhcp4_test.py
src/bin/dhcp6/tests/dhcp6_test.py

index 3ad97c6c98012e7cc17aa4389f1a2bd4cd373094..efe16fb7a909fcbbdf6afac36b7e0c8f16114e77 100644 (file)
@@ -32,11 +32,11 @@ class TestDhcpv4Daemon(unittest.TestCase):
         #
         # However, we do want to set the logging lock directory to somewhere
         # to which we can write - use the current working directory.  We then
-        # set the appropriate environment variable.
+        # set the appropriate environment variable.  os.putenv() doesn't work
+        # on Ubuntu, so we access os.environ directly.
         lockdir_envvar = "B10_LOCKFILE_DIR_FROM_BUILD"
-        lockdir = os.getenv(lockdir_envvar)
-        if lockdir is None:
-            os.putenv(lockdir_envvar, os.getcwd())
+        if lockdir_envvar not in os.environ:
+            os.environ[lockdir_envvar] = os.getcwd()
 
     def tearDown(self):
         pass
index 0887bbf2a0fa85ca78fc61034f3ee977a84940c7..78202bd910dedbdb866268d50df0b912352e4963 100644 (file)
@@ -32,11 +32,11 @@ class TestDhcpv6Daemon(unittest.TestCase):
         #
         # However, we do want to set the logging lock directory to somewhere
         # to which we can write - use the current working directory.  We then
-        # set the appropriate environment variable.
+        # set the appropriate environment variable.  os.putenv() doesn't work
+        # on Ubuntu, so we access os.environ directly.
         lockdir_envvar = "B10_LOCKFILE_DIR_FROM_BUILD"
-        lockdir = os.getenv(lockdir_envvar)
-        if lockdir is None:
-            os.putenv(lockdir_envvar, os.getcwd())
+        if lockdir_envvar not in os.environ:
+            os.environ[lockdir_envvar] = os.getcwd()
 
     def tearDown(self):
         pass