diff options
author | Mathew Robinson <mathew.robinson@mongodb.com> | 2020-01-16 23:04:43 +0000 |
---|---|---|
committer | evergreen <evergreen@mongodb.com> | 2020-01-16 23:04:43 +0000 |
commit | 02f3b3a204bdc0e1b157a3839be91dc56f685077 (patch) | |
tree | 578f9728ecbb082dce36169f3cdfacc22b3a8357 | |
parent | afca00af8b522304402b4671c7b9888776c0ec5f (diff) | |
download | mongo-02f3b3a204bdc0e1b157a3839be91dc56f685077.tar.gz |
SERVER-42702 $PYTHON should not be used in build signature
-rw-r--r-- | SConstruct | 2 | ||||
-rw-r--r-- | buildscripts/utils.py | 27 | ||||
-rwxr-xr-x | site_scons/site_tools/idl_tool.py | 2 |
3 files changed, 2 insertions, 29 deletions
diff --git a/SConstruct b/SConstruct index ee22daf3fdf..f65a4add436 100644 --- a/SConstruct +++ b/SConstruct @@ -1061,7 +1061,7 @@ envDict = dict(BUILD_ROOT=buildDir, MODULE_INJECTORS=dict(), ARCHIVE_ADDITION_DIR_MAP={}, ARCHIVE_ADDITIONS=[], - PYTHON=utils.find_python(), + PYTHON="$( {} $)".format(sys.executable), SERVER_ARCHIVE='${SERVER_DIST_BASENAME}${DIST_ARCHIVE_SUFFIX}', UNITTEST_ALIAS='unittests', # TODO: Move unittests.txt to $BUILD_DIR, but that requires diff --git a/buildscripts/utils.py b/buildscripts/utils.py index 4e2edcc76e5..8228675d084 100644 --- a/buildscripts/utils.py +++ b/buildscripts/utils.py @@ -130,33 +130,6 @@ def which(executable): return executable -def find_python(min_version=(3, 7)): - """Return path of python.""" - try: - return sys.executable - except AttributeError: - # In case the version of Python is somehow missing sys.version_info or sys.executable. - pass - - version = re.compile(r"[Pp]ython ([\d\.]+)", re.MULTILINE) - binaries = ("python37", "python3.7", "python36", "python3.6", "python35", "python3.5", "python") - for binary in binaries: - try: - out, err = subprocess.Popen([binary, "-V"], stdout=subprocess.PIPE, - stderr=subprocess.PIPE).communicate() - for stream in (out, err): - match = version.search(stream) - if match: - versiontuple = tuple(map(int, match.group(1).split("."))) - if versiontuple >= min_version: - return which(binary) - except Exception: # pylint: disable=broad-except - pass - - raise Exception( - "could not find suitable Python (version >= %s)" % ".".join(str(v) for v in min_version)) - - def replace_with_repr(unicode_error): """Codec error handler replacement.""" # Unicode is a pain, some strings cannot be unicode()'d diff --git a/site_scons/site_tools/idl_tool.py b/site_scons/site_tools/idl_tool.py index b041dc8cb7e..03b4254e417 100755 --- a/site_scons/site_tools/idl_tool.py +++ b/site_scons/site_tools/idl_tool.py @@ -94,7 +94,7 @@ def generate(env): global idlc idlc = idlc_mod - env["IDLC"] = sys.executable + " buildscripts/idl/idlc.py" + env["IDLC"] = "$PYTHON buildscripts/idl/idlc.py" env["IDLCFLAGS"] = "" base_dir = env.subst("$BUILD_ROOT/$VARIANT_DIR").replace("#", "") env["IDLCCOM"] = ( |