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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Daniel Stone <daniel@xxxxxxxxxxxxx> writes:

> Hi,
>
> On 27 January 2016 at 14:16, Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote:
>> On 27 January 2016 at 11:45, Daniel Stone <daniel@xxxxxxxxxxxxx> 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.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux