[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