On Mon, Dec 09, 2002, Michael Schloh wrote:
> 
>   Log:
>     Temporary workaround to fix fsl support.
> 
I believe this might be a flaw in fsl, though it mainly depends on what we
define as requiring --all. I can't think of a reason that we would want to
compile with the fsl library without linking in the libraries that fsl depends
on, so I would propose the following.

diff -Naur fsl.orig/fsl-config.in fsl/fsl-config.in
--- fsl.orig/fsl-config.in      Sat Jul 27 17:37:21 2002
+++ fsl/fsl-config.in   Mon Dec  9 20:12:19 2002
@@ -117,12 +117,10 @@
             output_extra="$output_extra $fsl_cflags"
             ;;
         --ldflags)
-            output="$output -L$fsl_libdir"
-            output_extra="$output_extra $fsl_ldflags"
+            output="$output -L$fsl_libdir $fsl_ldflags"
             ;;
         --libs)
-            output="$output -lfsl"
-            output_extra="$output_extra $fsl_libs"
+            output="$output -lfsl $fsl_libs"
             ;;
         * )
             echo "fsl-config:Error: Invalid option" 1>&2

If we decide that fsl should continue to indicate a link state which will fail
at build time, then it seems we will have to use the above workaround for all
fsl dependent packages.

-- 
[EMAIL PROTECTED]
Development Team, Application Services
Cable & Wireless Deutschland GmbH

[...]
>   Index: openpkg-src/sasl/sasl.spec
>   ============================================================
>   $ cvs diff -u -r1.36 -r1.37 sasl.spec
>   --- openpkg-src/sasl/sasl.spec      5 Dec 2002 12:52:31 -0000       1.36
>   +++ openpkg-src/sasl/sasl.spec      9 Dec 2002 18:58:16 -0000       1.37
>   @@ -41,7 +41,7 @@
>    Group:        Cryptography
>    License:      BSD
>    Version:      2.1.9
>   -Release:      20021205
>   +Release:      20021209
>    
>    #   list of sources
>    Source0:      ftp://ftp.andrew.cmu.edu/pub/cyrus-mail/cyrus-sasl-%{version}.tar.gz
>   @@ -90,8 +90,8 @@
>        CC="%{l_cc}" \
>        CFLAGS="%{l_cflags -O} -I%{l_prefix}/include" \
>    %if "%{with_fsl}" == "yes"
>   -    LDFLAGS="-L%{l_prefix}/lib `%{l_prefix}/bin/fsl-config --ldflags`" \
>   -    LIBS="-ldb `%{l_prefix}/bin/fsl-config --libs`" \
>   +    LDFLAGS="-L%{l_prefix}/lib `%{l_prefix}/bin/fsl-config --all --ldflags`" \
>   +    LIBS="-ldb `%{l_prefix}/bin/fsl-config --all --libs`" \
>    %else
>        LDFLAGS="-L%{l_prefix}/lib" \
>        LIBS="-ldb" \

Attachment: msg02835/pgp00000.pgp
Description: PGP signature

Reply via email to