> +OPTOUTSRC="$srcdir/src/optout.h"
> +
>  echo "
> -Update src/optout.h
> +Update $OPTOUTSRC
>  -------------------
>  Program compiled by: $USER@$ac_hostname"
>  
> -if ! test -e src/optout.h.bak; then
> -  cat src/optout.h > src/optout.h.bak
> +if ! test -e "$OPTOUTSRC"".bak" ; then
> +  cat "$OPTOUTSRC" > "$OPTOUTSRC"".bak"
>  fi
> -cat src/optout.h.bak | sed \
> +sed \
>    -e 's@#error@// &@' \
>    -e s/"FILL IN NAME HERE........................."/$USER@$ac_hostname/ \
> -  > src/optout.h
> +  < "$OPTOUTSRC"".bak" > "$OPTOUTSRC"

	Okay, I'll fix that, thanks !

> Instead, I get unresolved symbols from libpng. 
> The reason is that the script only looks for "png12".

	Yes, that particular point about libpng has been discussed
in the thread about the new configure script. I wanted to add some
tests about which version of libpng (if any) is present.

> -AC_CHECK_LIB([png12],[png_get_libpng_ver])
> +AC_SEARCH_LIBS([png_get_libpng_ver],[png12 png],,
> +       [AC_MSG_ERROR([Required PNG library (libpng) not detected.])])

	Okay I can add this, but I will have to check on our SGI machines
since they were the reason why I specified libpng12 instead of libpng.
But I suppose it will be okay if to search the lib in this order.


> Furthermore, one should probably take action if a library is not 
> found.

	Yeah that was discussed as well. I guess best would be to
change the code so that no graphics library is _required_ as long
as at least one image file format is supported on any platform.
In this case I guess libpng should be the required library, while
all the others could be optional. And all those #ifdef HAVE_lib
should be inserted where necessary. But that's not only UNIX-
specific, so I don't know if it will be accepted (at least soon)
by the POV-Team. Maybe it's already in the code afterall...

> Otherwise, nice configure script. 
> Can it be included in the standard UNIX source package?

	AFAIK Mark Gordon wanted to get some "inspiration" from it :-)
	All those mods cannot be directly included in the official
UNIX distro since there are a few GPL scripts provided in my package.

> [And src/conf.h removed because it it automatically generated and 
> breaks builds with srcdir!=builddir.]

	I'll check if it's also in my package. Otherwise I cannot
do much but deleting the file at configure time.

	I hope to upload a new version of the configure within this
week with some more extra architecture-oriented optimization flags.

	- NC

Post a reply to this message