setup.py: Be more restrictive with BST_CYTHON_TRACE valuesbschubert/fix-cython-trace

"0" evaluates to 'True' in python, which incorrectly switched on
the BST_CYTHON_TRACE. Forcing an int for this environment variable
allows us to ensure we are correct.

1 files changed, 6 insertions, 1 deletions

diff --git a/setup.py b/setup.py
index ad87932fb..ed211dd09 100755
--- a/setup.py
+++ b/setup.py
@@ -324,7 +324,12 @@ def assert_cython_required():
     raise SystemExit(1)
 
 
-ENABLE_CYTHON_TRACE = os.environ.get("BST_CYTHON_TRACE", "0")
+try:
+    ENABLE_CYTHON_TRACE = int(os.environ.get("BST_CYTHON_TRACE", "0"))
+except ValueError:
+    print("BST_CYTHON_TRACE must be an integer. Please set it to '1' to enable, '0' to disable", file=sys.stderr)
+    raise SystemExit(1)
+
 
 extension_macros = [
     ("CYTHON_TRACE", ENABLE_CYTHON_TRACE)