[PATCH] android: libdrm_platform: add liblog shared dependency

[Robert Foss]

On a second note, this does not apply on libdrm/master due
to:

LOCAL_SHARED_LIBRARIES := \
	libcutils

My thinking is that libcutils probably can be replaced with liblog,
but I'm not 100% sure.


Rob.

On 3/18/19 9:09 AM, Robert Foss wrote:
> This is probably a good idea!
> 
> 
> Reviewed-by: Robert Foss <robert.foss at collabora.com>
> 
> On 3/17/19 9:54 PM, Mauro Rossi wrote:
>> Hi,
>> I used the option --subject-prefix="PATCH libdrm"
>> but it did not go as expected.
>>
>> Anyway, the patch is for Android build of mesa/drm
>> Mauro
>>
>> On Sun, Mar 17, 2019 at 9:50 PM Mauro Rossi <issor.oruam at gmail.com> wrote:
>>>
>>> Fixes the following building error:
>>>
>>> FAILED: 
>>> $(OUT)/obj/SHARED_LIBRARIES/libdrm_platform_intermediates/LINKED/libdrm_platform.so 
>>>
>>> ...
>>> external/libdrm/xf86drm.c:146: error: undefined reference to 
>>> '__android_log_vprint'
>>> clang.real: error: linker command failed with exit code 1 (use -v to see 
>>> invocation)
>>>
>>> Signed-off-by: Mauro Rossi <issor.oruam at gmail.com>
>>> ---
>>>   Android.mk | 3 +++
>>>   1 file changed, 3 insertions(+)
>>>
>>> diff --git a/Android.mk b/Android.mk
>>> index f2c78bc1..f832b24e 100644
>>> --- a/Android.mk
>>> +++ b/Android.mk
>>> @@ -61,6 +61,9 @@ LOCAL_EXPORT_C_INCLUDE_DIRS := \
>>>          $(LOCAL_PATH) \
>>>          $(LOCAL_PATH)/include/drm
>>>
>>> +LOCAL_SHARED_LIBRARIES := \
>>> +       liblog
>>> +
>>>   LOCAL_C_INCLUDES := \
>>>          $(LOCAL_PATH)/include/drm
>>>
>>> -- 
>>> 2.19.1
>>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>