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

[Eric Anholt]

Daniel Stone <daniel at fooishbar.org> writes:

> Hi,
>
> On 27 January 2016 at 14:16, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 27 January 2016 at 11:45, Daniel Stone <daniel at fooishbar.org> wrote:
>>> 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.
>
> I suspect he was missing that 'Requires' also pulls in the relevant
> flags from those libraries, so PKG_CHECK_MODULES(DRM_VC4,
> [libdrm-vc4]) is equivalent to PKG_CHECK_MODULES(DRM_VC4, [libdrm
> libdrm-vc4]), if libdrm-vc4.pc has 'Requires: libdrm'.

What I heard Emil arguing to me was that a pkg-config request for --libs
From libdrm_vc4 shouldn't link against libdrm.  I'm fine with dropping
Libs and Cflags, as long as we still link libdrm like normal.  This all
seems like pointless bikeshedding, though.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20160127/adaf58c5/attachment.sig>