[PATCH] D38010: lit.py: Allow configs and local configs to have a setup_script entry

[Zachary Turner via Phabricator via llvm-commits]

zturner added inline comments.


================
Comment at: utils/lit/tests/Inputs/setup-script/subdir-custom/lit.local.cfg:1
+config.setup_script = 'Inputs/custom.sh'
----------------
zturner wrote:
> Can you make this not be a shell script?  This is essentially saying "This test doesn't work on Windows" when there's no particular reason why it shouldn't
Actually I take that back.  This //can't// be a shell script, because it's saying "this //feature// doesn't work on Windows".  So I think this has to be a Python script, and you just `execfile` it.  If you want to run a shell script, you can still do that by having a Python script which runs a shell script.  But the lit infrastructure should only deal with Python scripts.


Repository:
  rL LLVM

https://reviews.llvm.org/D38010