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

hhvm broke their xml support. ignore it until they can fix it #153

Merged
merged 1 commit into from Oct 21, 2014

Conversation

dbu
Copy link
Contributor

@dbu dbu commented Oct 21, 2014

stof added a commit that referenced this pull request Oct 21, 2014
hhvm broke their xml support. ignore it until they can fix it
@stof stof merged commit a28de77 into 1.0 Oct 21, 2014
@stof stof removed the in progress label Oct 21, 2014
@stof stof deleted the hhvm-broke-xml branch October 21, 2014 10:16
@dbu
Copy link
Contributor Author

dbu commented Oct 21, 2014

merged 1.0 into master to also have it there.

@fredemmott
Copy link

We do not consider it broken; the PHP5 behavior is a security issue, and can optionally be enabled in HHVM with an ini setting. This is explained in the error message.

dbu added a commit that referenced this pull request Nov 7, 2014
ddeboer added a commit that referenced this pull request Nov 7, 2014
revert #153 as hhvm setup on travis is fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants