[Nouveau] [libdrm 09/13] nouveau: import and install a selection of nvif headers from the kernel

Ben Skeggs skeggsb at gmail.com
Thu Nov 26 17:07:51 PST 2015


On 11/27/2015 07:14 AM, Emil Velikov wrote:
> On 26 November 2015 at 07:14, Ben Skeggs <skeggsb at gmail.com> wrote:
>> From: Ben Skeggs <bskeggs at redhat.com>
> 
>> --- 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
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20151127/30955814/attachment.sig>


More information about the Nouveau mailing list