On 11/27/2015 07:14 AM, Emil Velikov wrote:
> On 26 November 2015 at 07:14, Ben Skeggs <skeggsb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> From: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> 
>> --- a/nouveau/Makefile.am
>> +++ b/nouveau/Makefile.am
>> @@ -14,9 +14,18 @@ libdrm_nouveau_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
>>
>>  libdrm_nouveau_la_SOURCES = $(LIBDRM_NOUVEAU_FILES)
>>
>> -libdrm_nouveauincludedir = ${includedir}/libdrm
>> +libdrm_nouveauincludedir = ${includedir}/libdrm/nouveau
> Can we mention the change in the commit message, please ?
> 
> This will be a nice test for places missing LIBDRM_NOUVEAU_CFLAGS :-)
> I take it that neither mesa nor ddx explode with this change ?
Nope, they take it just fine.  I tested all sorts of combinations of
kernel (supports nvif vs not), libdrm (patched/unpatched), ddx (making
sure it builds), and mesa (patched/unpatched) to make sure that nothing
regressed.

Ben.

> 
> -Emil
>