[PATCH] D106941: test-release.sh: Kill python2

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 03:18:41 PDT 2021


rovka created this revision.
rovka added a reviewer: tstellar.
rovka requested review of this revision.
Herald added a project: LLVM.

Don't prefer python2's virtualenv when setting up the test-suite.
Always use python3 instead, since that's what we require everywhere else
anyway.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106941

Files:
  llvm/utils/release/test-release.sh


Index: llvm/utils/release/test-release.sh
===================================================================
--- llvm/utils/release/test-release.sh
+++ llvm/utils/release/test-release.sh
@@ -502,11 +502,8 @@
 # Setup the test-suite.  Do this early so we can catch failures before
 # we do the full 3 stage build.
 if [ $do_test_suite = "yes" ]; then
-  venv=virtualenv
-  if ! type -P 'virtualenv' > /dev/null 2>&1 ; then
-    check_program_exists 'python3'
-    venv="python3 -m venv"
-  fi
+  check_program_exists 'python3'
+  venv="python3 -m venv"
 
   SandboxDir="$BuildDir/sandbox"
   Lit=$SandboxDir/bin/lit


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106941.362314.patch
Type: text/x-patch
Size: 616 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210728/6ff51358/attachment.bin>


More information about the llvm-commits mailing list