[PATCH 3/3] vc4: Add headers and .pc files for VC4 userspace development.

[Emil Velikov]

On 27 January 2016 at 11:45, Daniel Stone <daniel at fooishbar.org> wrote:
> Hi,
>
> On 26 January 2016 at 20:10, Eric Anholt <eric at anholt.net> wrote:
>> Ilia Mirkin <imirkin at alum.mit.edu> writes:
>>> On Mon, Jan 25, 2016 at 2:27 PM, Eric Anholt <eric at anholt.net> wrote:
>>>> The headers were originally written in Mesa, imported to the kernel,
>>>> and improved upon in vc4-gpu-tools.  These come from the v-g-t copies
>>>> and will replace the Mesa and v-g-t copies, and hopefully be used from
>>>> new tests in igt, as well.
>>>>
>>>> Signed-off-by: Eric Anholt <eric at anholt.net>
>>>> ---
>>>> --- /dev/null
>>>> +++ b/vc4/libdrm_vc4.pc.in
>>>> @@ -0,0 +1,11 @@
>>>> +prefix=@prefix@
>>>> +exec_prefix=@exec_prefix@
>>>> +libdir=@libdir@
>>>> +includedir=@includedir@
>>>> +
>>>> +Name: libdrm_vc4
>>>> +Description: Userspace interface to vc4 kernel DRM services
>>>> +Version: @PACKAGE_VERSION@
>>>> +Requires: libdrm
>>>> +Libs: -L${libdir} -ldrm_intel
>>>
>>> This probably wants to be -ldrm_vc4? Or nothing if there's no such lib yet?
>>
>> Fixed to be -ldrm until we start building a libdrm_vc4.so.
>
> The Requires will take care of that, so you can just bin the entire
> 'Libs:' field until you need one:
>
In theory this will be sufficient, but Eric wasn't buying it [1] :'-(.
He's main consern (if I understood him correctly) is to stay
consistent with the existing libdrm_* providers, regardless if things
look a bit quirky.

-Emil

[1] http://people.freedesktop.org/~cbrill/dri-log/?channel=dri-devel&show_html=true&highlight_names=&date=2016-01-26
@13:22 onwards