[Bug 235293] Review Request: adminutil - Utility library for Fedora Directory Server administration

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: adminutil - Utility library for Fedora Directory Server administration
Alias: adminutil

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235293





------- Additional Comments From rmeggins at redhat.com  2007-05-24 14:53 EST -------
(In reply to comment #29)
> > But I thought since adminutil does not use any 
> > cyrus-sasl include files nor link
> > directly to any cyrus-sasl libs, it doesn't need 
> > BuildRequires: cyrus-sasl-devel?
> 
> Then please fix configure or the option of configre. 
> Actually the lastest tarball I have locally checks the existence of
> sasl.h and mock build fails without cyrus-sasl-devel.
> -----------------------------------------------------
> configure: checking for sasl...
> checking for --with-sasl... no
> checking for --with-sasl-inc... no
> checking for --with-sasl-lib... no
> checking for sasl.h... no
> configure: error: sasl not found, specify with --with-sasl.
> error: Bad exit status from /var/tmp/rpm-tmp.16648 (%build)

Ok.  This is the problem.  The actual package with just the shared libs does not
need sasl to build, because it doesn't build an executable.  However, a
developer may build tests, and since the tests are executables, they must be
linked against sasl:

gcc -g -O2 -o .libs/psetread tests/psetread-psetread.o  ./.libs/libadmsslutil.so
/share/adminutil/built/.libs/libadminutil.so -L/usr/lib64/dirsec -L/usr/lib64
./.libs/libadminutil.so -lplc4 -lplds4 -lnspr4 -lssl3 -lnss3 -lsoftokn3
-lssldap60 -lldap60 -lprldap60 -lldif60 -licui18n -licuuc -licudata  -Wl,--rpath
-Wl,/home/rich/11srv/lib
/usr/lib64/libldap60.so: undefined reference to `sasl_client_step'
...
/usr/lib64/libldap60.so: undefined reference to `sasl_encode'
collect2: ld returned 1 exit status
make: *** [psetread] Error 1

So I suppose the real solution here will be to add another configure option e.g.
--enable-tests.  Then I'll have to see if there is some way I can conditionally
include the m4/sasl.m4 file in configure.ac.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.