[Mesa-dev] [PATCH 3/7] anv/tests: Use a relative include path for anv_private.h

[Jason Ekstrand]

On Thu, Mar 2, 2017 at 12:13 PM, Jason Ekstrand <jason at jlekstrand.net>
wrote:

> On Thu, Mar 2, 2017 at 4:19 AM, Emil Velikov <emil.l.velikov at gmail.com>
> wrote:
>
>> On 2 March 2017 at 05:28, Jason Ekstrand <jason at jlekstrand.net> wrote:
>> > ---
>> >  src/intel/vulkan/tests/block_pool_no_free.c        | 2 +-
>> >  src/intel/vulkan/tests/state_pool.c                | 2 +-
>> >  src/intel/vulkan/tests/state_pool_free_list_only.c | 2 +-
>> >  src/intel/vulkan/tests/state_pool_no_free.c        | 2 +-
>> >  4 files changed, 4 insertions(+), 4 deletions(-)
>> >
>> > diff --git a/src/intel/vulkan/tests/block_pool_no_free.c
>> b/src/intel/vulkan/tests/block_pool_no_free.c
>> > index 86d1a76..4050f19 100644
>> > --- a/src/intel/vulkan/tests/block_pool_no_free.c
>> > +++ b/src/intel/vulkan/tests/block_pool_no_free.c
>> > @@ -23,7 +23,7 @@
>> >
>> >  #include <pthread.h>
>> >
>> > -#include "anv_private.h"
>> > +#include "../anv_private.h"
>> >
>> I've tried to prune most of these from the tree. Please pass the
>> corresponding -I to the compiler instead.
>>
>
> About that... I chose to go this route expressly to avoid having to add
> the -I.  If I add the -I, I either make everything in src/intel have
> -Isrc/intel/vulkan which seems rather silly, or I have to add a CPPFLAGS
> parameter for every single test.  I could do the latter but it makes for
> quite a bit of makefile bloat so I chose the former.  If you'd rather bloat
> the makefile, we can do that.
>

Never mind me.  Turns out I had to fix up CPPFLAGS anyway.  I'll drop this
patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170302/3a146769/attachment.html>