-
Notifications
You must be signed in to change notification settings - Fork 113
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
Get rid of libxml dependency #174
Comments
I see utf8_decode(), but preceded with function_exists() check, so composer.json should not require libxml. |
I think that here the intention was to require I will be happy to change it. Can you reproduce it somehow on travis ci? removing libxml, but keeping extdom? |
No, I didn't try with Travis, that came from one of Roundcube users. |
Actually, the ctype ticket came from the user. This one is from me just reviewing dependencies. |
I don't see anything in the code that would require libxml extension. We only need DOM. Do I miss something?
The text was updated successfully, but these errors were encountered: