Merging LDAP controls support
| From: | Côme Chilliet | Date: | Tue, 12 Sep 2017 15:39:51 +0000 |
| Subject: | Merging LDAP controls support | ||
| Groups: | php.internals | ||
| Request: | Send a blank email to internals+get-100543@lists.php.net to get a copy of this message | ||
Hello,
I’m getting close to have something I can merge in master for my PR adding support for LDAP
controls: https://github.com/php/php-src/pull/2640
In this case am I suppose to squash the changes in a single commit? This was suggested to me for a
previous PR but there is so much changes here I feel we should keep the history.
If some of you can pull my branch and launch the tests from ext/ldap it would help. You’ll need an
LDAP server and to fill the env vars from ext/ldap/tests/connect.inc.
Côme