Move the function strip_quotes to testutil.py
authorThomas Miedema <thomasmiedema@gmail.com>
Fri, 13 Mar 2015 20:07:15 +0000 (21:07 +0100)
committerThomas Miedema <thomasmiedema@gmail.com>
Fri, 13 Mar 2015 20:07:43 +0000 (21:07 +0100)
If one runs the testsuite with a profiling compiler, during the import
of `testlib.py`, `testlib.py` sets the global variable `gs_working`. To
do so, it executes a few statements which require the function
`strip_quotes` to be in scope. But that function only gets defined at
the very end of testlib.py.

This patch moves the definition of `strip_quotes` to testutil.py, which
is imported at the very top of testlib.py. This unbreaks the nightly
builders.

Reviewed By: austin

Differential Revision: https://phabricator.haskell.org/D728

testsuite/driver/testlib.py
testsuite/driver/testutil.py

index fec6939..1d3ef11 100644 (file)
@@ -2283,7 +2283,3 @@ def getStdout(cmd):
         return stdout
     else:
         raise Exception("Need subprocess to get stdout, but don't have it")
-
-def strip_quotes(s):
-    # Don't wrap commands to subprocess.call/Popen in quotes.
-    return s.strip('\'"')
index ec45e93..2cfa8f1 100644 (file)
@@ -15,3 +15,6 @@ def version_gt(x, y):
 def version_ge(x, y):
     return version_to_ints(x) >= version_to_ints(y)
 
+def strip_quotes(s):
+    # Don't wrap commands to subprocess.call/Popen in quotes.
+    return s.strip('\'"')