[Mesa-dev] [PATCH] drivers/x11: add gallium include dirs to Makefile.am
eric at anholt.net
Fri Feb 20 13:51:39 PST 2015
Matt Turner <mattst88 at gmail.com> writes:
> On Fri, Feb 20, 2015 at 12:20 PM, Brian Paul <brianp at vmware.com> wrote:
>> Fixes xlib driver build after e8c5cbfd921680c.
>> src/mesa/drivers/x11/Makefile.am | 2 ++
>> 1 file changed, 2 insertions(+)
>> diff --git a/src/mesa/drivers/x11/Makefile.am b/src/mesa/drivers/x11/Makefile.am
>> index 76a7cd4..c0596f8 100644
>> --- a/src/mesa/drivers/x11/Makefile.am
>> +++ b/src/mesa/drivers/x11/Makefile.am
>> @@ -30,6 +30,8 @@ AM_CPPFLAGS = \
>> -I$(top_srcdir)/src/mapi \
>> -I$(top_srcdir)/src/mesa \
>> -I$(top_srcdir)/src \
>> + -I$(top_srcdir)/src/gallium/include \
>> + -I$(top_srcdir)/src/gallium/auxiliary \
> I really dislike including these directories in a bunch of non-Gallium
> places. I should have said so more strongly when the patch went by.
> I'm not sure whether I want to spend my time to fix this the right way or what.
> I guess just go ahead and commit this until something better comes along.
I'm sympathetic, it's just the feeling of "if I can't get incremental
changes towards sharing, we'll never get anywhere and we'll be stuck
with this code duplication we've had for the last 5 years".
p_compiler.h looks like something we should be able to get out into
util/, at which point we could drop these include dirs. It's on my list
still, but I do really want to land my NIR work.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 818 bytes
Desc: not available
More information about the mesa-dev