On Sun, Apr 30, 2017 at 4:52 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> Hi Rob,
>
> On 27 April 2017 at 20:43, Rob Herring <r...@kernel.org> wrote:
>
>> --- a/src/gallium/targets/dri/Android.mk
>> +++ b/src/gallium/targets/dri/Android.mk
>> @@ -105,10 +105,9 @@ LOCAL_WHOLE_STATIC_LIBRARIES := \
>>         libmesa_util \
>>         libmesa_loader
>>
>> -LOCAL_STATIC_LIBRARIES :=
>> +LOCAL_STATIC_LIBRARIES := libelf
>>
>>  ifeq ($(MESA_ENABLE_LLVM),true)
>> -LOCAL_STATIC_LIBRARIES += libelf
> AFAICT we don't require any headers here, plus libelf is used only by
> the radeon drivers.
>
> Are you sure this hunk is required/correct?

Required, no, but neither is making it conditional either. It's just
to get rid of some of the conditionals.

Rob
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to