Hi Andreas,

> -----Original Message-----
> From: Andreas Heigl [mailto:andr...@heigl.org]
> Sent: Wednesday, June 24, 2015 5:40 PM
> To: internals@lists.php.net
> Cc: Côme BERNIGAUD
> Subject: [PHP-DEV] Deprecating ldap_sort
> 
> Hi everyone.
> 
> Côme Bernigaud and myself are currently cleaning up the LDAP-Extension
> (Well, Côme is doing the hard work and I'm trying to assist in some
> way). We would like to bring it in line with a more recent version of
> the OpenLDAP-lib. Currently the plan is to require OpenLDAP 2.4 as the
> minimum version to build ext/ldap against. This is on a very good way [1].
> 
> But in said OpenLDAP-library the ldap_sort-function already has been
> marked as deprecated [2]. Therefore we'd like to at least mark PHPs
> ldap_sort function as deprecated also.
> 
> The current rewrite will make it possible to later use the server-sided
> sort functionality so there will be only limited need for the current
> (client sided) ldap_sort function.
> 
> As it's a BC-break to remove the ldap_sort function will we have to
> setup an RFC for that? Or is it a plain "mark it deprecated in PHP7 and
> throw it away in PHP8" kind of decission? And will it be possible to get
> that marked deprecated in 7 at all?
> 
I've a few questions to this. Can it be implemented with non deprecated 
symbols? Or maybe, can the server side sort not be done with the same function, 
as it's probably the same job? Or it will be really not required? Any info 
about the plans on the openldap side to remove the deprecated symbols (AFAIR 
those are kept already for years)?

We're currently don't know, how wide this function is used and how much it 
would break. In general, deprecating it if there's a strong reason, could be 
sufficient. If there's a small possibility to keep this function, we should use 
it. Fe maybe it could kept and enabled with a configure option, that way it'll 
be still usable. 

Any feedback from the ldap users were appreciated here, as well.

Regards

Anatol


--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to