On 11/27/2015 07:14 AM, Emil Velikov wrote:
> On 26 November 2015 at 07:14, Ben Skeggs <[email protected]> wrote:
>> From: Ben Skeggs <[email protected]>
> 
>> --- 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
> 

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Nouveau mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to