[PATCH] D90291: [clangd] Add lit tests for remote index

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 2 06:28:50 PST 2020


hans added inline comments.


================
Comment at: clang-tools-extra/clangd/test/lit.site.cfg.py.in:11
 config.host_triple = "@LLVM_HOST_TRIPLE@"
+config.python_executable = "@Python3_EXECUTABLE@"
 
----------------
kadircet wrote:
> hans wrote:
> > Could this use `PYTHON_EXECUTABLE` instead? I see other tests using that, and Python3_EXECUTABLE might not always be set.
> i was checking llvm's [lit.site.cfg.py.in](http://github.com//llvm/llvm-project/tree/master/llvm/test/lit.site.cfg.py.in) for reference and it was using python3 too.
> 
> any specific reasons for using PYTHON_EXECUTABLE instead of python3? I can't say this was a strong preference, but the python script included might not be python2 compliant, hence the need for 3.
> 
> If there is a compelling reason for using a possibly python2 executable, we can try to make the script both 2 and 3 compliant.
> i was checking llvm's lit.site.cfg.py.in for reference and it was using python3 too.

Oh interesting, I hadn't seen that. Well then using it here too makes sense I suppose. Sorry for the noise :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90291



More information about the cfe-commits mailing list