From: Emil Velikov <emil.l.velikov@gmail.com>
To: Daniel Stone <daniel@fooishbar.org>
Cc: "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 3/3] vc4: Add headers and .pc files for VC4 userspace development.
Date: Wed, 27 Jan 2016 14:16:26 +0000 [thread overview]
Message-ID: <CACvgo50D98YTGMeA4XoAhq1JybSW0=aHD1ENniyMUL-m+ifTkA@mail.gmail.com> (raw)
In-Reply-To: <CAPj87rMYa2X-xphdaWu-4E=N16UimYktCYgqzG61UvH2e-N3iA@mail.gmail.com>
On 27 January 2016 at 11:45, Daniel Stone <daniel@fooishbar.org> wrote:
> Hi,
>
> On 26 January 2016 at 20:10, Eric Anholt <eric@anholt.net> wrote:
>> Ilia Mirkin <imirkin@alum.mit.edu> writes:
>>> On Mon, Jan 25, 2016 at 2:27 PM, Eric Anholt <eric@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@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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-01-27 14:16 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-25 19:27 [PATCH 1/3] vc4: Add the DRM header file Eric Anholt
2016-01-25 19:27 ` [PATCH 2/3] util: Add support for vc4 Eric Anholt
2016-01-25 19:27 ` [PATCH 3/3] vc4: Add headers and .pc files for VC4 userspace development Eric Anholt
2016-01-25 20:23 ` Ilia Mirkin
2016-01-26 20:10 ` Eric Anholt
2016-01-27 11:45 ` Daniel Stone
2016-01-27 14:16 ` Emil Velikov [this message]
2016-01-27 14:24 ` Daniel Stone
2016-01-28 3:04 ` Eric Anholt
2016-01-28 11:15 ` Emil Velikov
2016-01-28 3:24 ` [PATCH 3/3 v3] " Eric Anholt
2016-01-28 3:25 ` [PATCH] " Eric Anholt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CACvgo50D98YTGMeA4XoAhq1JybSW0=aHD1ENniyMUL-m+ifTkA@mail.gmail.com' \
--to=emil.l.velikov@gmail.com \
--cc=daniel@fooishbar.org \
--cc=dri-devel@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.