Re: Bug in configure

Hi John,

Jörg Sommer <[email protected]> wrote:
> there's a really, really ugly bug in configure:
>
> % sed -n 6838,+2p configure
>
>   if test X"jd_slang_include_dir" != X -a "$jd_slang_library_dir" != X
>                                          ^^^
>   then
>
> There's an X missing.
>
> diff --git a/autoconf/aclocal.m4 b/autoconf/aclocal.m4
> index 941c962..bb6e304 100644
> --- a/autoconf/aclocal.m4
> +++ b/autoconf/aclocal.m4
> @@ -890,7 +890,7 @@ AC_DEFUN(JD_CHECK_FOR_LIBRARY, dnl#{{{
>      fi
>    fi
>  
> -  if test X"jd_$1_include_dir" != X -a "$jd_$1_library_dir" != X
> +  if test X"jd_$1_include_dir" != X -a X"$jd_$1_library_dir" != X
>    then
>      AC_MSG_RESULT(yes: $jd_$1_library_dir and $jd_$1_include_dir)
>      jd_with_$1_library="yes"

You didn't update the aclocal file of slrn. You changed it for jed, but
not for slrn.

Bye, Jörg.
-- 
Real programmers don't comment their code.  It was hard to write,
it should be hard to understand.

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php