On 12-01-17 03:58 PM, Matt Turner wrote: > --- > src/mesa/drivers/dri/i965/Makefile.am | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/Makefile.am > b/src/mesa/drivers/dri/i965/Makefile.am > index 5512381..93937b1 100644 > --- a/src/mesa/drivers/dri/i965/Makefile.am > +++ b/src/mesa/drivers/dri/i965/Makefile.am > @@ -26,7 +26,7 @@ include Makefile.sources > # Hack to make some of the non-automake variables work. > TOP=$(top_builddir) > > -AM_CFLAGS = \ > +AM_CPPFLAGS = \ > -I$(top_srcdir)/include \ > -I$(top_srcdir)/src/ \ > -I$(top_srcdir)/src/mapi \ This looks suspicious and/or error prone. Are you sure that none of the variables assigned to AM_CPPFLAGS will ever contain any C compiler flags? While AM_CFLAGS can contain pre-processor flags, AM_CPPFLAGS cannot contain C compiler flags. I was not able to find where INTEL_CFLAGS is assigned, but as its name implies, it may contain C flags. If it is not used, it should be deleted. Automake has all the features to allow a user to append C flags on the make command line. > @@ -39,8 +39,6 @@ AM_CFLAGS = \ > $(API_DEFINES) \ > $(INTEL_CFLAGS) > > -AM_CXXFLAGS = $(AM_CFLAGS) > - > dridir = $(DRI_DRIVER_INSTALL_DIR) > > if HAVE_I965_DRI
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
