Edit report at http://bugs.php.net/bug.php?id=42060&edit=1
ID: 42060 Comment by: luisdv at icon dot co dot za Reported by: iarenuno at eteo dot mondragon dot edu Summary: [PATCH] LDAP: Add pagedResults support and more Status: Feedback Type: Feature/Change Request Package: *General Issues Operating System: * PHP Version: 5CVS, 6CVS (2008-11-01) Assigned To: pajoye Block user comment: N Private report: N New Comment: @jeanseb, thanks for compiling the dll. Is it for the Thread-Safe version of PHP only? I've tried running it on my WinXP SP3 box with PHP 5.2.14 VC6 Non-Thread-Safe but when I call PHPInfo() I get a message saying, "PHP Startup: Unable to load dynamic library 'C:\PHP\exe\php_ldap.dll' - The specified module could not be found." I've checked the C:\PHP\exe directory and the php_ldap.dll file (164kb) that you supplied is in that directory. When I restore the original php_ldap.dll file PHPInfo() runs fine and includes the LDAP section in the output. Previous Comments: ------------------------------------------------------------------------ [2010-11-23 22:37:12] sriram dot natarajan at gmail dot com I don't think this patch will be applied against 5.2 branch. since 5.2 release train is in security bug fix only. however, we can ask if this patch can be applied against 5.3. thanks for the patch. ------------------------------------------------------------------------ [2010-11-18 22:34:54] jeanseb at au-fil-du dot net Pierre ? PS : I have build the patch on PHP 5.2 : http://www.au-fil-du.net/public/php/php_ldap.PHP5.2.14-withPaginationSupport.Win32-VC6-x86.zip ------------------------------------------------------------------------ [2010-11-04 22:20:35] jeanseb at au-fil-du dot net That's it. You can apply ext-ldap-review.patch and api-rename.patch on trunk. For api-rename.patch, I should do some svn add & svn del on the phpt file. I have rename the 3 phpt but the svn diff seems to fail to create a good patch. ------------------------------------------------------------------------ [2010-11-03 23:04:53] jeanseb at au-fil-du dot net I have updated the initial patch (php-trunk_ldap-pagination.patch) to reflect this api change and i have integrated the commit of felipe @303159. Tomorrow I'll integrate my review patch (ext-ldap-review.patch) to the initial one and I'll rebuild one, based on trunk. ------------------------------------------------------------------------ [2010-11-03 13:20:46] paj...@php.net yes, please do, then I can apply the patch to trunk ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/bug.php?id=42060 -- Edit this bug report at http://bugs.php.net/bug.php?id=42060&edit=1
