Reverted. Thx for taking the time to argue with me :)<br><br><div class="gmail_quote">On Tue Nov 11 2014 at 2:44:14 AM NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I noticed this is the 2nd script that uses %python in llvm/test. (1st<br>
is BugPoint/compile-custom.ll)<br>
<br>
I don't understand your background. What happened to you?<br>
Could you update this to fit your environment? I am certain it'd be<br>
safe even on 2.4.<br>
<br>
Or, I hope you might work to disable this just on your environment.<br>
It's not good idea to disable this on other (expected-to-work) hosts.<br>
<br>
2014-11-11 1:07 GMT+09:00 Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>>:<br>
> Funnily enough it's not... (we're using our own lit runner).<br>
> If you think it's wrong I can revert and just black-list the test internally<br>
> though...<br>
><br>
> On Mon Nov 10 2014 at 5:05:45 PM NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>><br>
> wrote:<br>
>><br>
>> I assume python is required to build the tree.<br>
>><br>
>> 2014-11-11 1:03 GMT+09:00 Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>>:<br>
>> > I have a test environment that doesn't have python... I need some way to<br>
>> > mark this test to require python.<br>
>> ><br>
>> ><br>
>> > On Mon Nov 10 2014 at 4:46:50 PM NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> It should work with 2.6.6 and your "REQUIRES: python27" disables this<br>
>> >> around whole version of python. What was your issue?<br>
>> >><br>
>> >> 2014-11-11 0:29 GMT+09:00 Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>>:<br>
>> >> > Author: klimek<br>
>> >> > Date: Mon Nov 10 09:29:29 2014<br>
>> >> > New Revision: 221598<br>
>> >> ><br>
>> >> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=221598&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=221598&view=rev</a><br>
>> >> > Log:<br>
>> >> > Mark test using python as REQUIRES: python27.<br>
>> >> ><br>
>> >> > Modified:<br>
>> >> >     llvm/trunk/test/tools/llvm-<u></u>symbolizer/ppc64.test<br>
>> >> ><br>
>> >> > Modified: llvm/trunk/test/tools/llvm-<u></u>symbolizer/ppc64.test<br>
>> >> > URL:<br>
>> >> ><br>
>> >> > <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-symbolizer/ppc64.test?rev=221598&r1=221597&r2=221598&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/test/tools/<u></u>llvm-symbolizer/ppc64.test?<u></u>rev=221598&r1=221597&r2=<u></u>221598&view=diff</a><br>
>> >> ><br>
>> >> ><br>
>> >> > ==============================<u></u>==============================<u></u>==================<br>
>> >> > --- llvm/trunk/test/tools/llvm-<u></u>symbolizer/ppc64.test (original)<br>
>> >> > +++ llvm/trunk/test/tools/llvm-<u></u>symbolizer/ppc64.test Mon Nov 10<br>
>> >> > 09:29:29<br>
>> >> > 2014<br>
>> >> > @@ -1,3 +1,4 @@<br>
>> >> > +// REQUIRES: python27<br>
>> >> >  // ppc64 was compiled from this source on a big-endian 64-bit<br>
>> >> > PowerPC<br>
>> >> > box<br>
>> >> >  // with just "clang -nostdlib":<br>
>> >> >  int foo() { return 0; }<br>
>> >> ><br>
>> >> ><br>
>> >> > ______________________________<u></u>_________________<br>
>> >> > llvm-commits mailing list<br>
>> >> > <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
>> >> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>