[Cython] Fwd: [sage-devel] cython installation with python 2.3

[Stefan Behnel]

Hi,

in general, Cython should support 2.3, so it's a bug if it doesn't.

Michael Abshoff wrote:
> The first is in the file Cython/Compiler/TreeFragment.py, which in
> line 136 reads
> minindent = min(len(INDENT_RE.match(x).group(0)) for x in lines), but
> for my version of python has to be changed to
> minindent = min([len(INDENT_RE.match(x).group(0)) for x in lines])
> (explicit list comprehension). This should also work in newer versions
> of python.

Yep, that's already fixed in the developer version.


> The other is in setup.py, where lines 15-16 are
>    setup_args['data_files'] = [
>        {compiler_dir, ['Cython/Compiler/Lexicon.pickle']}]
> which I changed to
>    setup_args['data_files'] = [
>        (compiler_dir, ['Cython/Compiler/Lexicon.pickle'])]
> because I saw that (my installed version of) distutils expected a
> tuple, not a dictionary. I don't know if this would still work with
> later or other versions of distutils.

I checked, and that's what the Py2.3 docs tell me as well. This is code
specifically for Py2.3, so I have no idea how this could ever work...

Thanks for the report,
Stefan