[PATCH] D35909: [lit] Port googletest lit tests to Windows

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 26 12:00:29 PDT 2017


rnk created this revision.

The technique of directly calling subprocess.Popen on a python script
doesn't work on Windows. The executable path of the command must refer
to a valid win32 executable.

Instead, rename all the python scripts masquerading as gtest executables
to have .py extensions, so we can easily detect then and call the python
executable for them. Do this on Linux as well as Windows for
consistency.

Fixes PR33933


https://reviews.llvm.org/D35909

Files:
  llvm/utils/lit/lit/formats/googletest.py
  llvm/utils/lit/tests/Inputs/googletest-format/DummySubDir/OneTest
  llvm/utils/lit/tests/Inputs/googletest-format/DummySubDir/OneTest.py
  llvm/utils/lit/tests/Inputs/googletest-timeout/DummySubDir/OneTest
  llvm/utils/lit/tests/Inputs/googletest-timeout/DummySubDir/OneTest.py
  llvm/utils/lit/tests/Inputs/googletest-upstream-format/DummySubDir/OneTest
  llvm/utils/lit/tests/Inputs/googletest-upstream-format/DummySubDir/OneTest.py
  llvm/utils/lit/tests/googletest-format.py
  llvm/utils/lit/tests/googletest-timeout.py
  llvm/utils/lit/tests/googletest-upstream-format.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35909.108340.patch
Type: text/x-patch
Size: 10852 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170726/d7e53499/attachment.bin>


More information about the llvm-commits mailing list