[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