[PATCH] D69684: [debuginfo-tests] Don't look for Python 3 if we already have it

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 7 17:32:46 PST 2019


aprantl added a comment.

(green dragon uses an all-python3 configuration now)

In D69684#1737796 <https://reviews.llvm.org/D69684#1737796>, @rnk wrote:

> Would it be OK if we removed all the logic to redo the python search here, and replace it with a simple version check? So far as I know, most of the rest of LLVM is Python 3 ready, so even if you use python 2 on PATH, there's no reason not to tell cmake to use Python 3.


I would very much prefer a hard error over anything automagic. I just wanted to make sure no-one else ( @jmorse ) relies on this functionality.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69684/new/

https://reviews.llvm.org/D69684





More information about the llvm-commits mailing list