Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support libxml2 2.9.0 and later #986

Closed
knu opened this issue Oct 26, 2013 · 8 comments · Fixed by #1182
Closed

Support libxml2 2.9.0 and later #986

knu opened this issue Oct 26, 2013 · 8 comments · Fixed by #1182

Comments

@knu
Copy link
Member

knu commented Oct 26, 2013

No description provided.

@knu
Copy link
Member Author

knu commented Nov 1, 2013

Looking at #829 and https://bugzilla.gnome.org/show_bug.cgi?id=695699 it seems libxml2 2.9.x is completely unusable for us.

I'll add some code to extconf.rb so it won't pick libxml2 2.9.0 or later for build.

@knu
Copy link
Member Author

knu commented Nov 1, 2013

Well, I mean 2.9.0 and 2.9.1. We'll revisit when libxml2 2.9.2 or whatever comes out.

@knu
Copy link
Member Author

knu commented Nov 2, 2013

I tested Nokogiri with the current libxml2 master and the number of failures were just two. Seems the position() problem is gone.

@knu
Copy link
Member Author

knu commented Nov 5, 2013

Just started a libxml2-2.9.1 branch in preparation for the next release of libxml2.

@postmodern
Copy link

Looking forward to 2.9.1 support. Noticed nokogiri kept printing a warning about being built against 2.8.0, but loading 2.9.1. Oddly, I do not have libxml 2.8.0 installed on my system.

@flavorjones
Copy link
Member

@knu, I'd like to work on this today and tomorrow. Have you made any progress since the Nov 4 commit on that branch? If not, no problem, I just don't want to reproduce work you've already possibly done.

@knu
Copy link
Member Author

knu commented Aug 22, 2014

@flavorjones Unfortunately not much, so please go ahead and tackle it. I'll be following up whenever I find something. Thanks!

@postmodern
Copy link

🎆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants