On Mon, Feb 18, 2013 at 7:58 PM, Rob Clark <robdcl...@gmail.com> wrote: > On Mon, Feb 18, 2013 at 12:47 PM, Matt Turner <matts...@gmail.com> wrote: >> On Sun, Feb 17, 2013 at 11:33 AM, Rob Clark <robdcl...@gmail.com> wrote: >>> >>> diff --git a/src/gallium/drivers/freedreno/Makefile.am >>> b/src/gallium/drivers/freedreno/Makefile.am >>> new file mode 100644 >>> index 0000000..10abdfb >>> --- /dev/null >>> +++ b/src/gallium/drivers/freedreno/Makefile.am >>> @@ -0,0 +1,35 @@ >>> +include $(top_srcdir)/src/gallium/Automake.inc >>> + >>> +noinst_LTLIBRARIES = libfreedreno.la >>> + >>> +AM_CFLAGS = \ >>> + -Werror -Wno-packed-bitfield-compat \ >>> + -I$(top_srcdir)/src/gallium/include \ <-- >>> + -I$(top_srcdir)/src/gallium/auxiliary \ <-- >>> + -I$(top_srcdir)/src/gallium/drivers \ >>> + -I$(top_srcdir)/include \ <-- >>> + $(FREEDRENO_CFLAGS) \ >>> + $(DEFINES) \ <-- >>> + $(PIC_FLAGS) \ >>> + $(VISIBILITY_CFLAGS) >> >> The <-- mark things that are provided by the GALLIUM_CFLAGS variable >> in Automake.inc that you've already included. PIC_FLAGS is now dead. >> Distributions don't like -Werror being hardcoded into upstream's >> CFLAGS. > > Hmm, is there a better way to get -Werror for just freedreno when I am > building myself? I do find that it is pretty useful to let the > compiler help me catch problems rather than debugging them the hard > way ;-)
You can set the CFLAGS and CXXFLAGS environment variables before configuring Mesa. Marek _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
