Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (athena.apache.org: local policy)
Message-Id: <4790BF72.3235.00AC.0@novell.com>
Date: Fri, 18 Jan 2008 14:59:37 -0700
From: "Brad Nicholes" <bnicholes@novell.com>
To: "APR dev list." <dev@apr.apache.org>,
	"Paul J. Reder" <rederpj@remulak.net>
Subject: [Fwd: Re: [Patch]: Add a couple of cases to the
	apr_ldap_set_option function.]
References: <4790D699.6030008@remulak.net>
In-Reply-To: <4790D699.6030008@remulak.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Committed.


>>> On Fri, Jan 18, 2008 at  9:40 AM, in message <4790D699.6030008@remulak.=
net>,
"Paul J. Reder" <rederpj@remulak.net> wrote:=20

> Any takers?
>=20
> I'm still hoping to get this reviewed by someone and have it committed =
to
> APR (since I don't have karma for that) so I can commit the Apache =
portion
> to trunk.
>=20
> Thanks,
>=20
> Paul J. Reder
>=20
> -------- Original Message --------
> Subject: [Fwd: Re: [Patch]: Add a couple of cases to the apr_ldap_set_opt=
ion=20
> function.]
> Date: Wed, 19 Dec 2007 22:41:10 -0500
> From: Paul J. Reder <rederpj@remulak.net>
> To: APR dev list. <dev@apr.apache.org>
>=20
> Just in case it would help in evaluating these APR changes this is the
> link to the Apache-trunk code, which uses these APR changes, that I'm
> waiting to commit...
>=20
> http://people.apache.org/~rederpj/Apache-trunk_rebind.diff
>=20
> Thanks,
>=20
> Paul J. Reder
>=20
> -------- Original Message --------
> Subject: Re: [Patch]: Add a couple of cases to the apr_ldap_set_option=20=

> function.
> Date: Fri, 14 Dec 2007 18:57:17 -0500
> From: Paul J. Reder <rederpj@remulak.net>
> To: APR dev list. <dev@apr.apache.org>
> References: <47604914.5030007@remulak.net>
>=20
> Brad confirmed that this should work for Novell. My pass through the
> limited Solaris docs I was able find seemed to indicate this should =
work.
>=20
> http://people.apache.org/~rederpj/apr-util-ldap.diff
>=20
> Any more reviews? Anyone to commit it?
>=20
> I was hoping to get this committed so I could commit the Apache trunk
> code that uses this before I go on vacation next week.
>=20
> Thanks again,
>=20
> Paul J. Reder wrote:
>> In keeping with the APR concept of hiding LDAP differences, I added =
a=20
>> couple
>> of cases to the apr_ldap_set_option function. One of the cases has=20
>> differences
>> between at least the openLDAP and Tivoli SDKs so I added both of them =
for
>> consistency and to handle possible differences in other SDKs.
>>=20
>> Folks who know the other SDKs should check if any #ifs need to be added
>> to these cases for any of the other SDKs.
>>=20
>> Once someone commits this patch I will be able to commit the Apache =
portion
>> of the rebind code.