Re: [PATCH 26/29] mpv: add PACKAGECONFIG for drm/gbm/vdpau/wayland

["Andreas Müller" <schnitzeltony@googlemail.com>]

On Thu, Apr 13, 2017 at 11:46 AM, Andrey Konovalov
<andrey.konovalov@linaro.org> wrote:
> On 04/12/2017 10:56 PM, Andreas Müller wrote:
>>
>> Signed-off-by: Andreas Müller <schnitzeltony@googlemail.com>
>> ---
>>   meta-oe/recipes-multimedia/mplayer/mpv_0.24.0.bb | 13 +++++++++++--
>>   1 file changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta-oe/recipes-multimedia/mplayer/mpv_0.24.0.bb
>> b/meta-oe/recipes-multimedia/mplayer/mpv_0.24.0.bb
>> index 3f46dc1..60f653a 100644
>> --- a/meta-oe/recipes-multimedia/mplayer/mpv_0.24.0.bb
>> +++ b/meta-oe/recipes-multimedia/mplayer/mpv_0.24.0.bb
>> @@ -3,7 +3,7 @@ DESCRIPTION = "mpv is a fork of mplayer2 and MPlayer. It
>> shares some features wi
>>   SECTION = "multimedia"
>>   HOMEPAGE = "http://www.mpv.io/"
>>   DEPENDS = "zlib ffmpeg jpeg virtual/libx11 xsp libxv \
>> -           libxscrnsaver libv4l libxinerama libvdpau \
>> +           libxscrnsaver libv4l libxinerama \
>>   "
>>
>>   REQUIRED_DISTRO_FEATURES = "x11"
>> @@ -24,14 +24,23 @@ SRC_URI[waf.sha256sum] =
>> "01bf2beab2106d1558800c8709bc2c8e496d3da4a2ca343fe091f2
>>   inherit waf pkgconfig pythonnative distro_features_check
>>
>>   # Note: both lua and libass are required to get on-screen-display
>> (controls)
>> -PACKAGECONFIG ??= "lua libass"
>> +PACKAGECONFIG ??= " \
>> +    lua \
>> +    libass \
>> +    ${@bb.utils.filter('DISTRO_FEATURES', 'wayland', d)} \
>> +"
>> +PACKAGECONFIG[drm] = "--enable-drm,--disable-drm,libdrm"
>> +PACKAGECONFIG[gbm] = "--enable-gbm,--disable-gbm,virtual/mesa"
>>   PACKAGECONFIG[lua] = "--enable-lua,--disable-lua,lua luajit"
>>   PACKAGECONFIG[libass] = "--enable-libass,--disable-libass,libass"
>>   PACKAGECONFIG[libarchive] =
>> "--enable-libarchive,--disable-libarchive,libarchive"
>>   PACKAGECONFIG[jack] = "--enable-jack, --disable-jack, jack"
>>   PACKAGECONFIG[vaapi] = "--enable-vaapi, --disable-vaapi,libva"
>> +PACKAGECONFIG[vdpau] = "--enable-vdpau, --disable-vdpau,libvdpau"
>> +PACKAGECONFIG[wayland] = "--enable-wayland, --enable-wayland,wayland"
>
>
> Should it be:
> PACKAGECONFIG[wayland] = "--enable-wayland, --disable-wayland,wayland"
> ?
Yes - Thanks - needs V2

Andreas