New configure

[Vin Shelton]

"Stephen J. Turnbull" <stephen at xemacs.org> writes:

>>>>>> "vin" == Vin Shelton <acs at xemacs.org> writes:
>
>     vin> I don't know what you had in mind.  Here's the configure info
>     vin> from today's 21.5 build:
>
> configure:22636: checking for png_read_image in -lpng
> configure:22666: icc -o conftest   -Wall -w1 -we147 -g -O3 -Ob2  -Os -tpp6 -axK -xK -ip   -I/usr/local/include     -I/usr/X11R6/include    -z nocombreloc  -L/usr/local/lib    -L/usr/X11R6/lib  conftest.c    -ljpeg -lz -lXpm -lXmu -lXt -lXext -lX11   -lSM -lICE -lpng      1>&5
>
> It's not that -lz "comes before" -lpng; that part is actually
> irrelevant to the way autoconf works.  Autoconf simply wasn't told
> that PNG needs zlib.

It's not irrelevant to the linker, however.  If you replace the
literal string "-lpng" in the test above with "-lpng -lz", that test
will pass and png will be found.  That's what my --with-libs=-lz test
was about.

>
> Try this patch (according to the logic for zlib, TIFF needs it too):
>
> Index: ChangeLog
> ===================================================================
> RCS file: /pack/xemacscvs/XEmacs/xemacs/ChangeLog,v
> retrieving revision 1.444
> diff -u -U0 -r1.444 ChangeLog
> --- ChangeLog	14 Mar 2005 12:22:16 -0000	1.444
> +++ ChangeLog	15 Mar 2005 03:14:37 -0000
> @@ -0,0 +1,4 @@
> +2005-03-15  Stephen J. Turnbull  <stephen at xemacs.org>
> +
> +	* configure.ac (Autodetect PNG): PNG & TIFF need -lz in AC_CHECK_LIB.
> +
>
> Index: configure.ac
> ===================================================================
> RCS file: /pack/xemacscvs/XEmacs/xemacs/configure.ac,v
> retrieving revision 1.3
> diff -u -r1.3 configure.ac
> --- configure.ac	14 Mar 2005 12:22:22 -0000	1.3
> +++ configure.ac	15 Mar 2005 03:14:44 -0000
> @@ -3811,7 +3811,7 @@
>    png_problem=""
>    test -z "$with_png" && { AC_CHECK_FUNC(pow,                  ,with_png=no) }
>    test -z "$with_png" && { AC_CHECK_HEADER(png.h,              ,with_png=no) }
> -  test -z "$with_png" && { AC_CHECK_LIB(png, png_read_image,[:],with_png=no) }
> +  test -z "$with_png" && { AC_CHECK_LIB(png, png_read_image,[:],with_png=no,[-lz]) }
>    if test -z "$with_png"; then
>      AC_MSG_CHECKING(for workable png version information)
>      xe_check_libs="-lpng -lz"
> @@ -3847,7 +3847,7 @@
>  
>    dnl autodetect TIFF
>    test -z "$with_tiff" && { AC_CHECK_HEADER(tiffio.h,            ,with_tiff=no) }
> -  test -z "$with_tiff" && { AC_CHECK_LIB(tiff, TIFFClientOpen,[:],with_tiff=no) }
> +  test -z "$with_tiff" && { AC_CHECK_LIB(tiff, TIFFClientOpen,[:],with_tiff=no,[-lz]) }
>    test -z "$with_tiff" && with_tiff=yes
>    if test "$with_tiff" = "yes"; then
>      AC_DEFINE(HAVE_TIFF)
>


Yup, that gets it.  Thank you, Stephen.

   - Vin