-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Add PHP 7.1 drone tests #836
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
|
@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @MorrisJobke and @rullzer to be potential reviewers |
|
Ref guzzle/guzzle#1393 for the Guzzle error that upstream exists. But since we updated Guzzle it may be already fixed. |
Details |
|
phpseclib/phpseclib#1013 has the phpseclib patches |
|
phpseclib/phpseclib@15c9f99 is the breaking phpseclib commit |
|
PHP 5.4 fails: Details |
|
As #836 (comment) says 😉 – This is caused by an change in phpseclib behaviour. Need to think about how to handle this… |
.htaccess
Outdated
| #### DO NOT CHANGE ANYTHING ABOVE THIS LINE #### | ||
|
|
||
| ErrorDocument 403 /core/templates/403.php | ||
| ErrorDocument 404 /core/templates/404.php |
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.
meeeep
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.
Ah... bad autotests.sh touching that file all the time.
|
@rullzer Is this ready for review? |
|
Nope it is an ugly ugly work around. But i have ideas on how to clean up a bit.. Will do soonish |
|
Ok ready for reviews. See https://tools.ietf.org/html/rfc3447#page-38 for the salt length of 0. |
|
LGTM |
* Set salt length to 0 See https://tools.ietf.org/html/rfc3447#page-38 * Fix tests for new phpunit
|
Rebased on top of merged 3rdparty and fixed a merge conflict |
Bumps the PHP version from beta2 to RC1
|
Failing on the updated PHP 7.1 RC1, the errors are strange though. Mhm. 🙈 |
|
Seems like JSON_PRETTY_PRINT behaves differently now… |
In php 7.1 the pretty print json output changed. Thus now we compare arrays.
|
Looks good 👍 |
|
🎉 |
Build should fail. But we can then step by step add fixes and merge when green :)
cc @MorrisJobke