-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[2.3] Fix phpunit test suite on travis #10824
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
Conversation
romainneutron
commented
Apr 30, 2014
Q | A |
---|---|
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | #10821 |
License | MIT |
@@ -1,4 +1,4 @@ | |||
<?xml version="1.0" ?> | |||
<?xml version="1.0" encoding="utf-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some tests were failing because of this, see https://travis-ci.org/romainneutron/symfony/jobs/24088967
@webmozart We have a test that fails on PHP 5.5 and 5.6, it also fails on my computer (PHP 5.5.11) see https://travis-ci.org/symfony/symfony/builds/24090008
That's weird as none of the test or the listener have been changed for a while. |
@romainneutron this is because 4.0.19 was not yet released. |
I suggest using #10826 instead as downloading a phar will be faster than running a composer installation |
@stof what about your comment about openssl (#10826 (comment)), #10826 bypasses tests on 5.3 |
yeah, I saw it after commented here. |
@loicfrering do you think it's possible to add openssl to PHP 5.3.3 recipe? |
closed in favor of #10826 |
@romainneutron we had to drop openssl on 5.3.3 due to compilation issues with the recent versions of OpenSSL. |