[PATCH v2 video-vmware] Do not use per-target compilation flags
Gaetan Nadon
memsize at videotron.ca
Mon Oct 21 23:22:52 CEST 2013
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.
Reviewed-By: Jakob Bornecrantz <jakob at vmware.com>
Signed-off-by: Gaetan Nadon <memsize at videotron.ca>
---
*** Original version was reviewed by Jakob, Thomas found vmwgfx dir
was missing. Added same changes to vmwgfx in v2.
configure.ac | 1 -
saa/Makefile.am | 2 +-
src/Makefile.am | 2 +-
vmwgfx/Makefile.am | 2 +-
4 files changed, 3 insertions(+), 4 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
diff --git a/vmwgfx/Makefile.am b/vmwgfx/Makefile.am
index 269d870..fb33730 100644
--- a/vmwgfx/Makefile.am
+++ b/vmwgfx/Makefile.am
@@ -1,7 +1,7 @@
if BUILD_VMWGFX
noinst_LTLIBRARIES = libvmwgfx.la
-libvmwgfx_la_CFLAGS = $(CWARNFLAGS) $(XORG_CFLAGS) @LIBDRM_CFLAGS@ @XATRACKER_CFLAGS@ -I$(top_srcdir)/src -I$(top_srcdir)/saa
+AM_CFLAGS = $(CWARNFLAGS) $(XORG_CFLAGS) @LIBDRM_CFLAGS@ @XATRACKER_CFLAGS@ -I$(top_srcdir)/src -I$(top_srcdir)/saa
libvmwgfx_la_LIBADD = @LIBDRM_LIBS@ $(top_builddir)/saa/libsaa.la\
@XATRACKER_LIBS@
libvmwgfx_la_DEPENDENCIES = $(top_builddir)/saa/libsaa.la
--
1.7.9.5
More information about the xorg-devel
mailing list