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@xxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review