[sane-devel] Re: Problem compiling 1.0.7 on HPUX 10.20
Henning Meier-Geinitz
henning at meier-geinitz.de
Sun Feb 3 10:20:01 GMT 2002
Hi,
On Sat, Feb 02, 2002 at 09:47:56PM +0100, Michael Piotrowski wrote:
> I just compiled 1.0.7beta2 on HP-UX 10.20; if you're using the HP C
> compiler, the flags described in README.hpux are still correct:
>
> CC=cc CFLAGS="-Ae -O" ./configure
Thanks. So I will add this to the list of supported platforms for 1.0.7.
> However, if you're using gcc, you now have to specify -D_HPUX_SOURCE
> (to get the definition of struct stat), e.g.:
>
> CC=gcc CFLAGS="-D_HPUX_SOURCE -O" ./configure
>
> This is necessary, because (in contrast to earlier versions) gcc is
> now called with the -ansi option, which undefines the symbols which
> control the inclusion of "beyond-C89" definitions.
>
> I couldn't find where this is happening, but since config.h already
> contains:
That's what I mentioned in my previous mail: All the releases are
compiled with "--disable-warnings" while the CVS and beta versions are
compiled with "--enable-warnings" by default. So the "-ansi" can be
found in the configure.in file.
Can you try make distclean and ./configure --disable-warnings or
better, try my latest snapshot at
http://www.meier-geinitz.de/sane/tmp/sane-backends-1.0.7-cvs.tar.gz
This one has the warnings already disabled.
> #if defined (__sun) && defined (__GNUC__)
> # define _POSIX_SOURCE
> # define __EXTENSIONS__
> #endif
>
> I would suggest adding something like:
>
> #if defined (__hpux) && defined (__GNUC__)
> # define _HPUX_SOURCE
> #endif
Or we just disable -ansi for HPUX in configure. I'll think about it
for CVS.
> Unfortunately, at least in umax_pp_low.c, sys/time.h is included
> before sane/config.h, which results in another problem (struct timeval
> isn't defined), so maybe adding -D_HPUX_SOURCE to the compiler options
> when gcc is used is safer and easier.
That's a bug in umax_pp_low. This will be fixed after the
release because with "--disable-warnings" compilation should work.
I don't want to introduce new #include order bugs that short before
release.
Thanks for testing.
Bye,
Henning
More information about the sane-devel
mailing list