Re: ldap.set_option() ready

[Michael <mi...@st...>]

Jacek Konieczny wrote:
> 
> On Tue, Nov 13, 2001 at 10:13:54PM +1000, David Leonard wrote:
> > On Tue, 13 Nov 2001, Jacek Konieczny typed thusly:
> >
> > > The two interfaces can coexist.
> >
> > um, i'd like to avoid too many ways of doing the same thing.. it leads to
> > confusion... so, just one way or the other thanks :)
> 
> In this case IMHO set_option() seems more reasonable.

Go for it.

> But it will break
> backward compatibility (for those options which are already implemented
> as attributes).

Backward compability to python-ldap with OpenLDAP 1 libs regarding
options was broken with OpenLDAP 2 patches anyway. I don't mind.

> I won't touch __setattr__() and set_option() of LDAPObject until we
> decide on some solution.

Go for set_option() since the C module part should map OpenLDAP 2
API more or less directly.

Ciao, Michael.