[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: SLAPI (ITS#2073)
Ando, -- Thank you for looking the code over. I will be making a new
submission within the next few days that correct some defects. I am
suspecting the libtool warning comes from the fact I am linking static
libraries (it is more than just libtmpslapd.a) to produce dynamic library
(libslapi.so). I don not like the Warning either, however, in testing I did
not experience any problem. In our build environment, we do not see this
warning. Suggestions are welcome.
Thanks,
Steve Omrani
Senior Software Developer
Internet: somrani@us.ibm.com
Phone: 512-838-8375 (TL 678-8375)
Fax: 512-838-8597
"Pierangelo
Masarati" To: Steve Omrani/Austin/IBM@IBMUS
<ando@sys-net.it> cc: <openldap-its@openldap.org>
Subject: Re: SLAPI (ITS#2073)
11/04/2002 03:02
AM
Please respond to
ando
Steve,
I've uploaded a patch to
http://www.sys-net.it/~ando/Download/Pierangelo-Masarati-20021104-Steve-Omrani.ns_slapi_.diff
which includes the porting of your submission
to current HEAD code and some fixes. Among these
you should carefully review "slapi_ops.c" because
there is a number of pointers that are used
uninitialized (significantly the pointer to the
Backend structure, "be"). Other fixes are related
to slapi API functions, like the (apparently)
inconsistent use of slapi_strlen()/slapi_ch_stlen()
(none of which is defined). I also get very scary
warnings from libtool because you statically link
some libtmpslapd.a to libslapi (if I get it correctly)
in an unportable manner.
Note that I didn't check the functionality of SLAPI
itself (this will require a little more); I only
verified that I can compile OpenLDAP safely and
that the usual tests run OK.
In case you think my changes are appropriate, send
me a note (or a patch with further fixes) and I'll
commit everything.
Of course, any other comment or review is welcome
before I actually commit the patch.
Pierangelo.
--
Pierangelo Masarati
mailto:pierangelo.masarati@sys-net.it