Jean-Marc Lasgouttes wrote:
> Angus> When did I post this mail? Feels like ages ago!
> 
> Gnus tells me: 7 weeks, 2 days, 8 hours, 42 minutes, 16 seconds ago
> (although the 16 seconds part may not be  accurate by the time you
> read this).

;-) (We seem to be forgetting the no smiley Friday rule.)

>>> Concerning the X flags, I agree they should not be there.
> 
> Angus> Is this an autotools problem or could we augment the INCLUDES
> Angus> variable in the Makefile.ams? Looks to me like X includes
> Angus> should be needed only in the xforms, qt2 and gtk directories
> Angus> and in the last two only if compiling on a posix system.
> 
> The code in configure.ac says:
> 
> ### Check for X libraries
> AC_PATH_XTRA
> case $have_x in
>   yes) LIBS="$X_PRE_LIBS $LIBS $X_LIBS -lX11 $X_EXTRA_LIBS"
>        CPPFLAGS="$CPPFLAGS $X_CFLAGS";;
> esac
> 
> The setting of CPPFLAGS is obviously not good, but the code for the
> frontend relies on having everything available, I guess.

Sure, but couldn't the individual Makefile.ams have something like

INCLUDES = $(BOOST_INCLUDES) @X_CFLAGS@

so only those Makefile.ams that actually need the X includes use 'em?

-- 
Angus

Reply via email to