<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 29, 2016 at 8:47 AM, Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Nov 29, 2016 at 08:28:55AM -0800, Jason Ekstrand wrote:<br>
> On Tue, Nov 29, 2016 at 1:23 AM, <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a><wbr>> wrote:<br>
><br>
> > From: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a><wbr>><br>
> ><br>
> > Set the include paths to consider in-tree headers before out-of-tree<br>
> > headers.<br>
> ><br>
> > Avoids the build failing due to stale headers being present in<br>
> > $prefix. Previosuly 'make -ki install' or something similar was required<br>
> > to update the out-of-tree headers to allow the build to succeed.<br>
> ><br>
> > Also avoids having to rebuild the entire thing after every 'make<br>
> > install'.<br>
> ><br>
> > Cc: Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>><br>
> > Cc: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br>
> > Signed-off-by: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a><wbr>><br>
> > ---<br>
> >  src/intel/vulkan/Makefile.am | 16 +++++++++++-----<br>
> >  1 file changed, 11 insertions(+), 5 deletions(-)<br>
> ><br>
> > diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am<br>
> > index ce31abb2fce7..b80f8767296c 100644<br>
> > --- a/src/intel/vulkan/Makefile.am<br>
> > +++ b/src/intel/vulkan/Makefile.am<br>
> > @@ -43,9 +43,6 @@ noinst_LTLIBRARIES = $(PER_GEN_LIBS)<br>
> >  # The gallium includes are for the util/u_math.h include from<br>
> > main/macros.h<br>
> ><br>
> >  AM_CPPFLAGS = \<br>
> > -       $(INTEL_CFLAGS) \<br>
> > -       $(VALGRIND_CFLAGS) \<br>
> > -       $(DEFINES) \<br>
> >         -I$(top_srcdir)/include \<br>
> >         -I$(top_builddir)/src \<br>
> >         -I$(top_srcdir)/src \<br>
> > @@ -61,6 +58,17 @@ AM_CPPFLAGS = \<br>
> >         -I$(top_builddir)/src/intel \<br>
> >         -I$(top_srcdir)/src/intel<br>
> ><br>
> > +if HAVE_PLATFORM_WAYLAND<br>
> > +AM_CPPFLAGS += \<br>
> > +       -I$(top_builddir)/src/egl/<wbr>wayland/wayland-drm \<br>
> > +       -I$(top_srcdir)/src/egl/<wbr>wayland/wayland-drm<br>
> > +endif<br>
> ><br>
><br>
> I think I have a mild preference for keeping wayland stuff together and<br>
> moving the last AM_CPPFLAGS down but I don't care that much.  Either way,<br>
<br>
</div></div>Would we still want all internal -I knobs to appear before any<br>
external ones? To do that I'd have flip the x11 vs. wayland stuff<br>
around. And if someone were to add internal -I knobs for x11 then<br>
this scheme wouldn't work either way.<br></blockquote><div><br></div><div>Oh... I dind't think about that interaction.  Go with the way you had it for now.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Not sure if sticking to a strict global ordering like that is needed,<br>
but at least it would seem a bit easier to maintain as you wouldn't<br>
have to think too hard when adding new flags.<br></blockquote><div><br></div><div>We could have two things and then combine them later but that seems a bit painful.  Feel free to ignore my comments. :)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
><br>
> Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
><br>
><br>
> > +<br>
> > +AM_CPPFLAGS += \<br>
> > +       $(INTEL_CFLAGS) \<br>
> > +       $(VALGRIND_CFLAGS) \<br>
> > +       $(DEFINES)<br>
> > +<br>
> >  AM_CFLAGS = \<br>
> >         $(VISIBILITY_CFLAGS) \<br>
> >         -Wno-override-init -msse2<br>
> > @@ -99,8 +107,6 @@ endif<br>
> ><br>
> >  if HAVE_PLATFORM_WAYLAND<br>
> >  AM_CPPFLAGS += \<br>
> > -       -I$(top_builddir)/src/egl/<wbr>wayland/wayland-drm \<br>
> > -       -I$(top_srcdir)/src/egl/<wbr>wayland/wayland-drm \<br>
> >         $(WAYLAND_CFLAGS) \<br>
> >         -DVK_USE_PLATFORM_WAYLAND_KHR<br>
> ><br>
> > --<br>
> > 2.7.4<br>
> ><br>
> > ______________________________<wbr>_________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
> ><br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Ville Syrjälä<br>
Intel OTC<br>
</font></span></blockquote></div><br></div></div>