[debuginfo-tests] 7035ea6 - [dexter] Remove lit check for python 3
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 5 10:50:03 PST 2019
Author: Reid Kleckner
Date: 2019-11-05T10:49:56-08:00
New Revision: 7035ea6e3e4371fff223a3a8f660ec149a954876
URL: https://github.com/llvm/llvm-project/commit/7035ea6e3e4371fff223a3a8f660ec149a954876
DIFF: https://github.com/llvm/llvm-project/commit/7035ea6e3e4371fff223a3a8f660ec149a954876.diff
LOG: [dexter] Remove lit check for python 3
This is checking the version of Python used to run lit, which is not
necessarily the same as the version used to run the dexter tests. If
the tests are run via the build/bin/llvm-lit[.py] helper script, then
that is likely to pick up whatever version of Python is on PATH.
Conventionally, this will find Python 2. CMake already checks that
Python 3 is in use and puts the path to it in the lit site config, so
this check is redundant, and Python 3 will ultimately be used to run
dexter.
Reviewers: jmorse
Differential Revision: https://reviews.llvm.org/D69724
Added:
Modified:
debuginfo-tests/dexter/feature_tests/lit.local.cfg
Removed:
################################################################################
diff --git a/debuginfo-tests/dexter/feature_tests/lit.local.cfg b/debuginfo-tests/dexter/feature_tests/lit.local.cfg
index 39ead87cf3c5..5e9b7908f8b6 100644
--- a/debuginfo-tests/dexter/feature_tests/lit.local.cfg
+++ b/debuginfo-tests/dexter/feature_tests/lit.local.cfg
@@ -1,8 +1,2 @@
config.name = "DExTer feature tests"
config.suffixes = ['.cpp', '.c', '.test']
-
-# Dexter requires python3
-
-import sys
-if sys.version_info.major < 3:
- config.unsupported = True
More information about the llvm-commits
mailing list