Would it be more robust to interpret the shebang ourselves?  The current method relies on future script writers to know that it must end in .py.  We could interpret the shebang ourselves though and run Python if it's present <br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 26, 2017 at 12:19 PM Brian Gesiak via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">modocache requested changes to this revision.<br>
modocache added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
LGTM, but I want to double-check this passes for me locally. Also, I think you may have left some debugging code in here.<br>
<br>
<br>
<br>
================<br>
Comment at: llvm/utils/lit/lit/formats/googletest.py:92<br>
                 # Discover the tests in this executable.<br>
+                litConfig.warning(subdir)<br>
                 execpath = os.path.join(source_path, subdir, fn)<br>
----------------<br>
My guess is that you may have added this for your own debugging? Or is this intentional?<br>
<br>
<br>
================<br>
Comment at: llvm/utils/lit/tests/googletest-format.py:9<br>
 # CHECK: -- Testing:<br>
-# CHECK: PASS: googletest-format :: DummySubDir/OneTest/FirstTest.subTestA<br>
-# CHECK: FAIL: googletest-format :: DummySubDir/OneTest/FirstTest.subTestB<br>
-# CHECK-NEXT: *** TEST 'googletest-format :: DummySubDir/OneTest/FirstTest.subTestB' FAILED ***<br>
+# CHECK: PASS: googletest-format :: {{[Dd]ummy[Ss]ub[Dd]ir}}/OneTest.py/FirstTest.subTestA<br>
+# CHECK: FAIL: googletest-format :: {{[Dd]ummy[Ss]ub[Dd]ir}}/OneTest.py/FirstTest.subTestB<br>
----------------<br>
This is interesting, why do we need to handle upper- and lower-case here? Could you explain this in the commit message?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D35909" rel="noreferrer" target="_blank">https://reviews.llvm.org/D35909</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>