Require older lxml version for OSX compatibility

[syllog1sm]

Currently, the package does not work on OSX, if installed directly via pip, because "pip install lxml" does not currently work on OSX.

I don't know how many programmer-hours have been lost due to frustrations in installing lxml on OSX, due to OSX shipping out-dated libxml libraries. StackOverflow is littered with questions about it, and even the solutions that worked for other people didn't work for me.

What did work for me to use this package was to simply install an older lxml. If you write a requirements.txt in the root of this package, and write the lxml dependency as e.g.

lxml==2.3

The package will work successfully on OSX. It works up to lxml 2.3.5, and doesn't work with lxml 3.0. I'm not sure what the oldest version your package will work with; I haven't tested that sorry.

P.S Thanks for your work on this package. If this wasn't here, I'd have probably written it myself.

P.P.S. After spending hours on this stupid, stupid problem, let me just say: **** developing on OSX sucks.

[buriy]

Thanks a lot for this message, I believe now it will be easier for other users to google the correct solution to the problem (when they will see this page).

Generally, I suggest to install lxml manually. Also, on OS X, homebrew and virtualenv are better ways to use python.

[buriy]

Added conditional dependency in readability-lxml v0.3

[thinxer]

I don't know about other Mac OS X versions. With Mavericks (10.9) I can now successfully compile the lastest lxml library with pip.

The old lxml 2.3 library has some bugs that affects my code.

[buriy]

Thanks, I'll try now on my 10.9 and update requirements to check for OS version as well.

[buriy]

On 10.9.2, newest lxml can be installed with
export CFLAGS=-Qunused-arguments
export CPPFLAGS=-Qunused-arguments
sudo -E pip install --upgrade lxml
It seems to be working fine with readability-lxml.
Thanks!

[thinxer]

Yeah that's it! Basically everything needs compiling in pip will need that
flag in Mavericks :D.
Thanks for the quick reply.

On Wednesday, April 2, 2014, Yuri Baburov notifications@github.com wrote:

On 10.9.2, newest lxml can be installed with
export CFLAGS=-Qunused-arguments
export CPPFLAGS=-Qunused-arguments
sudo -E pip install --upgrade lxml
It seems to be working fine with readability-lxml.
Thanks!

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/39#issuecomment-39295652
.