[PATCH xserver] config: conftest needs an include directive to locate X11/Xfuncproto.h

[Alan Coopersmith]

On 11/30/11 15:54, Gaetan Nadon wrote:
> The compiler option -fvisibility=hidden is erroneously missing
> due to a faulty configuration test. The gcc command is unable to
> locate X11/Xfuncproto.h unless the build occurs on a
> system where X11 development headers are installed.
>
> configure:21294: checking for symbol visibility support
> configure:21323: gcc -std=gnu99 -c -g -O2 -fvisibility=hidden  conftest.c>&5
> conftest.c:144:28: fatal error: X11/Xfuncproto.h: No such file or directory
>
> The solution is to add an include directive to obtain the location
> of X11/Xfuncproto.h which may or may not be the system installed headers.
>
> Signed-off-by: Gaetan Nadon<memsize at videotron.ca>
> ---
>   configure.ac |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index e4fcba4..3d5414f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1558,7 +1558,8 @@ if test "x$XORG" = xyes; then
>   	    fi
>   	    if test x$have_visibility != xno; then
>   		save_CFLAGS="$CFLAGS"
> -		CFLAGS="$CFLAGS $VISIBILITY_CFLAGS"
> +		proto_inc=`$PKG_CONFIG --cflags xproto`
> +		CFLAGS="$CFLAGS $VISIBILITY_CFLAGS $proto_inc"
>   		AC_TRY_COMPILE(
>   		    [#include<X11/Xfuncproto.h>
>   		extern _X_HIDDEN int hidden_int;

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System