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.

Sign up for GitHub

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

Jump to bottom

Add setup check for libxml2 >= 2.7.0 #23168

Closed
rullzer opened this issue Mar 11, 2016 · 3 comments · Fixed by #23189
Closed

Add setup check for libxml2 >= 2.7.0 #23168

rullzer opened this issue Mar 11, 2016 · 3 comments · Fixed by #23189
Assignees
@LukasReschke
Labels
1 - To develop enhancement settings:admin settings:installation sev4-low
Milestone
9.1

Comments

@rullzer
Copy link
Contributor

rullzer commented Mar 11, 2016

As shown in #23054 (comment) we require at least libxml2-2.7.0.

I suspect this to be already the case for some time. But in 9.0 we switched to webdav also for the webUI.

Note libxml2-2.7.0 was released in august 2008. So it should not be a common issues.

Would be good to add a check for this in the setup warning.
@rullzer rullzer added this to the 9.1-current milestone Mar 11, 2016
@LukasReschke LukasReschke self-assigned this Mar 12, 2016
LukasReschke added a commit that referenced this issue Mar 12, 2016
@LukasReschke
Require at least libxml 2.7.0
42a70ed 
Fixes #23168
@LukasReschke LukasReschke mentioned this issue Mar 12, 2016
Merged
@PVince81
Copy link
Contributor

I think Sabre 3.0 which we use in OC 9.0 is using a different XML approach, using some PHP XML methods. I guess these might be using libxml internally somehow...

@rullzer
Copy link
Contributor Author

rullzer commented Mar 14, 2016

Yes. https://groups.google.com/d/msg/sabredav-discuss/9JxxSr7mGBY/gEM52ghVCgAJ

LukasReschke added a commit that referenced this issue Mar 16, 2016
@LukasReschke
Require at least libxml 2.7.0
554066b 
Fixes #23168
LukasReschke added a commit that referenced this issue Mar 16, 2016
@LukasReschke
Require at least libxml 2.7.0
dbcb037 
Fixes #23168
@LukasReschke LukasReschke mentioned this issue Mar 16, 2016
Merged
@ghost ghost mentioned this issue Apr 19, 2016
Closed
@lock
Copy link

lock bot commented Aug 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@LukasReschke LukasReschke

Labels
1 - To develop enhancement settings:admin settings:installation sev4-low
Projects
None yet
Milestone
9.1
Development

Successfully merging a pull request may close this issue.

Require at least libxml 2.7.0 owncloud/core
4 participants
@rullzer @PVince81 @LukasReschke @DeepDiver1975