[PATCH] libXt: util: don't link makestrs with target cflags

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon Nov 19 14:46:43 PST 2012


Hello,

Anyone for this basic libXt patch?

Thanks,

Thomas

On Fri, 16 Nov 2012 10:41:06 +0100, Thomas Petazzoni wrote:
> The line:
> 
> AM_CFLAGS = $(XT_CFLAGS)
> 
> in util/Makefile.am is wrong because it adds target cflags to the
> compilation of makestrs, which is built for the build machine, which
> leads to build failures when cross-compiling.
> 
> We also remove the inclusion of <X11/Xos.h> from makestrs.c, because
> it was the only non-standard header being included (and therefore
> possibly requiring special cflags), but it was in reality not useful
> at all to build makestrs.c.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
>  util/Makefile.am |    1 -
>  util/makestrs.c  |    1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/util/Makefile.am b/util/Makefile.am
> index dedfa6b..cc6f3fc 100644
> --- a/util/Makefile.am
> +++ b/util/Makefile.am
> @@ -10,7 +10,6 @@ EXTRA_DIST = \
>  	StrDefs.ht \
>  	string.list
>  
> -AM_CFLAGS = $(XT_CFLAGS)
>  makestrs_SOURCES = makestrs.c
>  
>  
> diff --git a/util/makestrs.c b/util/makestrs.c
> index a52866a..00c861f 100644
> --- a/util/makestrs.c
> +++ b/util/makestrs.c
> @@ -27,7 +27,6 @@ in this Software without prior written authorization from The Open Group.
>  /* Constructs string definitions */
>  
>  #include <stdio.h>
> -#include <X11/Xos.h>
>  #include <stdlib.h>
>  #include <unistd.h>
>  



-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com


More information about the xorg-devel mailing list