[PATCH app-xkbcomp] config: move CWARNFLAGS from configure.ac to Makefile.am
Alan Coopersmith
Alan.Coopersmith at Sun.COM
Thu Feb 11 17:09:47 PST 2010
I've been doing that as I do other updates, but hadn't finished doing them all
yet, so thanks for finishing.
Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
Gaetan Nadon wrote:
> Compiler warning flags should be explicitly set in the makefile
> rather than being merged with other packages compiler flags.
>
> Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
> ---
> Makefile.am | 2 +-
> configure.ac | 1 -
> 2 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/Makefile.am b/Makefile.am
> index 15ddc27..3550c40 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -21,7 +21,7 @@
>
> bin_PROGRAMS = xkbcomp
>
> -AM_CFLAGS = $(XKBCOMP_CFLAGS) -DDFLT_XKB_CONFIG_ROOT='"$(XKBCONFIGROOT)"'
> +AM_CFLAGS = $(XKBCOMP_CFLAGS) -DDFLT_XKB_CONFIG_ROOT='"$(XKBCONFIGROOT)"' $(CWARNFLAGS)
> xkbcomp_LDADD = $(XKBCOMP_LIBS)
>
> xkbcomp_SOURCES = \
> diff --git a/configure.ac b/configure.ac
> index 51457fc..4c86fcd 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -42,7 +42,6 @@ AC_CHECK_FUNCS([strdup strcasecmp])
>
> # Checks for pkg-config packages
> PKG_CHECK_MODULES(XKBCOMP, x11 xkbfile)
> -XKBCOMP_CFLAGS="$CWARNFLAGS $XKBCOMP_CFLAGS"
> AC_SUBST(XKBCOMP_CFLAGS)
> AC_SUBST(XKBCOMP_LIBS)
>
--
-Alan Coopersmith- alan.coopersmith at sun.com
Sun Microsystems, Inc. - X Window System Engineering
More information about the xorg-devel
mailing list