On Wed, Sep 17, 2014 at 07:11:25PM -0700, Jason Gerecke wrote: > Clang by default does not emit warnings for system headers, but > most includes will be added with '-I' which doesn't count as a > system header. This patch has them included with '-isystem' so > that we can more easily see problems with *our* code. > > Signed-off-by: Jason Gerecke <killert...@gmail.com> > --- > configure.ac | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/configure.ac b/configure.ac > index 6faf58c..d6de0ff 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -53,12 +53,15 @@ XPROTOS="xproto xext kbproto inputproto randrproto" > > # Obtain compiler/linker options from server and required extensions > PKG_CHECK_MODULES(XORG, [xorg-server >= 1.7.0] $XPROTOS) > +XORG_CFLAGS=`echo $XORG_CFLAGS | sed 's/-I/-isystem /'` > > # Obtain compiler/linker options for the xsetwacom tool > PKG_CHECK_MODULES(X11, x11 xi xrandr xinerama $XPROTOS) > +X11_CFLAGS=`echo $X11_CFLAGS | sed 's/-I/-isystem /'`
libX11 headers are set in stone, but server headers and udev can be fixed. I'd rather use the error messages you got to fix the server headers than simply silencing them. NAK to this one from me, sorry. Cheers, Peter > > # Obtain compiler/linker options for libudev used by ISDV4 code > PKG_CHECK_MODULES(UDEV, libudev) > +UDEV_CFLAGS=`echo $UDEV_CFLAGS | sed 's/-I/-isystem /'` > > # X Server SDK location is required to install wacom header files > # This location is also relayed in the xorg-wacom.pc file > -- > 2.1.0 > ------------------------------------------------------------------------------ Slashdot TV. Video for Nerds. Stuff that Matters. http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk _______________________________________________ Linuxwacom-devel mailing list Linuxwacom-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel
