[lxml-dev] lxml now ships with Pyrex included

[Stefan Behnel]

Hi,

Martijn Faassen wrote:
> Stefan Behnel wrote:
>> Now that this works, we could also remove the .c files to shrink the
>> source distribution. Any objections?
> 
> Yes, I object to removing the .c files. the .c files allow a guaranteed
> sure build of lxml. There is absolutely no doubt which version of Pyrex
> is used here, as that's under our own control (otherwise there might be
> interesting import issues in play).

I won't argue for it and it's fine to leave them in. It's only some 230K
difference in the tgz (50% more), so if that prevents us from running into
install problems ...


> Also important, it allows tools like easy_install to download and
> compile lxml fully automatically (such as in a buildout). I use this
> feature all the time. I don't know whether that can work if a Pyrex is
> bundled - the setup.py would likely become more complicated to account
> for PYTHONPATH manipulation, and that might possibly break easy_install.

No changes in setup.py are required. I only added Pyrex' package directory to
the lxml root directory and setup.py imports it nicely from there. I tested
that it builds with setuptools, so I don't quite see where this could
interfere with buildout or easy_install.

Stefan