[llvm] r224129 - Require python 2.7.

[Rafael Espíndola]

I have emailed a patch adding a cmake check. I will ping it.


On 25 December 2014 at 17:34, Hal Finkel <hfinkel at anl.gov> wrote:
> ----- Original Message -----
>> From: "Rafael Espíndola" <rafael.espindola at gmail.com>
>> To: "Paul Robinson" <Paul_Robinson at playstation.sony.com>
>> Cc: llvm-commits at cs.uiuc.edu
>> Sent: Friday, December 12, 2014 1:05:24 PM
>> Subject: Re: [llvm] r224129 - Require python 2.7.
>>
>> On 12 December 2014 at 13:42, Robinson, Paul
>> <Paul_Robinson at playstation.sony.com> wrote:
>> > Is there corresponding cmake work to do?
>>
>> Looks like cmake doesn't enforce the minimum python version. Not sure
>> if we would be better off if we started enforcing on the cmake side
>> or
>> if we dropped the autoconf check.
>
> I think we should check this in cmake too. TestCases/Posix/asan-symbolize-sanity-test.cc fails if I run the tests using a python 2.6 install with this error:
>
> Traceback (most recent call last):
>   File "/path/to/llvm/projects/compiler-rt/lib/asan/scripts/asan_symbolize.py", line 10, in <module>
>     import argparse
> ImportError: No module named argparse
>
> , but works fine with a 2.7 build, so enforcing 2.7 at configure time seems like a good idea.
>
>  -Hal
>
>>
>> Cheers,
>> Rafael
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory