[PATCH video-vmware] Do not use per-target compilation flags

Jakob Bornecrantz wallbraker at gmail.com
Mon Sep 30 14:45:24 PDT 2013


Looks good, thanks.

On Mon, Sep 30, 2013 at 8:51 PM, Gaetan Nadon <memsize at videotron.ca> wrote:
> Using "vmware_drv_la_CFLAGS" triggers the per-target compilation flag feature
> which requires AM_PROG_CC_C_O automake macro. The implementation of this feature
> is changing in automake 1.14 such that the macro is obsolete.
>
> Given that using this per-target feature is not needed, might as well remove
> it's usage from the driver configuration all together. The module will
> configure with no warnings on any version of automake.
>
> Note that none of he video drivers use this feature.
>
> Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
> ---
>  configure.ac    |    1 -
>  saa/Makefile.am |    2 +-
>  src/Makefile.am |    2 +-
>  3 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index dccfb27..b1f429b 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -46,7 +46,6 @@ AC_DISABLE_STATIC
>  AC_PROG_LIBTOOL
>
>  # Checks for programs.
> -AM_PROG_CC_C_O
>
>  AH_TOP([#include "xorg-server.h"])
>
> diff --git a/saa/Makefile.am b/saa/Makefile.am
> index 48c9734..890a246 100644
> --- a/saa/Makefile.am
> +++ b/saa/Makefile.am
> @@ -2,7 +2,7 @@
>  if BUILD_VMWGFX
>  noinst_LTLIBRARIES = libsaa.la
>
> -libsaa_la_CFLAGS = $(CWARNFLAGS) $(XORG_CFLAGS)
> +AM_CFLAGS = $(CWARNFLAGS) $(XORG_CFLAGS)
>  libsaa_la_LDFLAGS = -static
>  libsaa_la_SOURCES = \
>         saa.c \
> diff --git a/src/Makefile.am b/src/Makefile.am
> index b0dd147..1d12fda 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -24,9 +24,9 @@
>  # _ladir passes a dummy rpath to libtool so the thing will actually link
>  # TODO: -nostdlib/-Bstatic/-lgcc platform magic, not installing the .a, etc.
>
> +AM_CFLAGS = $(CWARNFLAGS) @XORG_CFLAGS@
>  vmware_drv_la_LTLIBRARIES = vmware_drv.la
>  vmware_drv_la_LDFLAGS = -module -avoid-version
> -vmware_drv_la_CFLAGS = $(CWARNFLAGS) @XORG_CFLAGS@
>  vmware_drv_ladir = @moduledir@/drivers
>
>  if BUILD_VMWGFX
> --
> 1.7.9.5
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel


More information about the xorg-devel mailing list