From: Jacob Stiffler <j-stiffler@ti.com>
To: Denys Dmytriyenko <denis@denix.org>, <meta-arago@arago-project.org>
Subject: Re: [PATCH] gstreamer, conf: add GST_PROVIDER to switch between 0.10 and 1.x gstreamer
Date: Mon, 27 Apr 2015 14:50:28 -0400 [thread overview]
Message-ID: <553E84F4.5080403@ti.com> (raw)
In-Reply-To: <1430157196-11756-1-git-send-email-denis@denix.org>
Most of this looks good to me.
However, in the previous patch to update glsdk-example-apps, there was a
SRCREV bump which is missing here. Is the newer SRCREV required for
gstreamer1.0? And would it be compatible with gstreamer 0.10?
Thanks,
Jake
On 4/27/2015 1:53 PM, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
> meta-arago-distro/conf/distro/arago.conf | 9 +++++---
> .../conf/distro/include/arago-prefs.inc | 8 ++++---
> .../packagegroup-arago-gst-sdk-target.bb | 16 ++++++++++++--
> .../packagegroups/packagegroup-arago-gst.bb | 25 ++++++++++++++++------
> .../glsdk-example-apps/glsdk-example-apps.bb | 4 ++--
> 5 files changed, 46 insertions(+), 16 deletions(-)
>
> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
> index 450ffe7..5debe0b 100644
> --- a/meta-arago-distro/conf/distro/arago.conf
> +++ b/meta-arago-distro/conf/distro/arago.conf
> @@ -24,6 +24,9 @@ DISTRO_FEATURES = "alsa argp bluetooth ext2 irda largefile pcmcia usbgadget usbh
> # Set additional distro features
> DISTRO_FEATURES += "pam ${@base_contains('MACHINE_FEATURES', 'screen', 'wayland', '', d)}"
>
> +# Set preferred gstreamer version (0.10 vs 1.0)
> +GST_PROVIDER ?= "1.0"
> +
> # Set preferred Qt version
> QT_PROVIDER ?= "qt5"
>
> @@ -35,8 +38,8 @@ PACKAGECONFIG_pn-weston = "fbdev kms launch libinput"
> PACKAGECONFIG_GL_pn-qtbase = "gles2 linuxfb"
> PACKAGECONFIG_DISTRO_pn-qtbase = "icu examples"
>
> -# Configure gstreamer-0.10.x as a dependency instead of gstreamer-1.x
> -PACKAGECONFIG_pn-qtwebkit = "gstreamer010 qtlocation qtmultimedia qtsensors"
> +# Configure gstreamer dependency based on GST_PROVIDER
> +PACKAGECONFIG_pn-qtwebkit = "${@base_conditional('GST_PROVIDER', '0.10', 'gstreamer010', 'gstreamer', d)} qtlocation qtmultimedia qtsensors"
>
> # Enable UI and scripting for perf
> PERF_FEATURES_ENABLE ?= "perf-scripting perf-tui"
> @@ -86,4 +89,4 @@ ENTERPRISE_DISTRO = "1"
> # disable by default the codecs in libav that causes issues with shipping an
> # Arago based SDK. No need to prevent gst-ffmpeg from being used since we
> # disable troublesome software.
> -LICENSE_FLAGS_WHITELIST = "commercial_gst-ffmpeg"
> +LICENSE_FLAGS_WHITELIST = "commercial_gst-ffmpeg commercial_gstreamer1.0-libav"
> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> index 4f2c854..9f12562 100644
> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> @@ -113,9 +113,11 @@ PREFERRED_PROVIDER_wpa-supplicant_ti33x = "wpa-supplicant-wl18xx"
> PREFERRED_PROVIDER_wpa-supplicant_ti43x = "wpa-supplicant-wl18xx"
> PREFERRED_PROVIDER_wpa-supplicant_omap-a15 = "wpa-supplicant-wl18xx"
>
> -PREFERRED_PROVIDER_gstreamer = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gstreamer-mm-accel', 'gstreamer', d)}"
> -PREFERRED_PROVIDER_gstreamer-dev = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gstreamer-mm-accel-dev', 'gstreamer-dev', d)}"
> -PREFERRED_PROVIDER_gstreamer-dbg = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gstreamer-mm-accel-dbg', 'gstreamer-dbg', d)}"
> +GST_MM_ACCEL = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gstreamer-mm-accel', 'gstreamer', d)}"
> +PREFERRED_PROVIDER_gstreamer = "${@base_conditional('GST_PROVIDER', '0.10', "${GST_MM_ACCEL}", 'gstreamer', d)}"
> +PREFERRED_PROVIDER_gstreamer-dev = "${@base_conditional('GST_PROVIDER', '0.10', "${GST_MM_ACCEL}-dev", 'gstreamer-dev', d)}"
> +PREFERRED_PROVIDER_gstreamer-dbg = "${@base_conditional('GST_PROVIDER', '0.10', "${GST_MM_ACCEL}-dbg", 'gstreamer-dbg', d)}"
> +
> PREFERRED_PROVIDER_gst-plugins-bad = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gst-plugins-bad-mm-accel', 'gst-plugins-bad', d)}"
> PREFERRED_PROVIDER_gst-plugins-bad-dev = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gst-plugins-bad-mm-accel-dev', 'gst-plugins-bad-dev', d)}"
> PREFERRED_PROVIDER_gst-plugins-bad-meta = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gst-plugins-bad-mm-accel-meta', 'gst-plugins-bad-meta', d)}"
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> index 7897b8a..37e63a3 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst-sdk-target.bb
> @@ -1,13 +1,25 @@
> DESCRIPTION = "Task to build and install header and libs in sdk"
> LICENSE = "MIT"
> -PR = "r7"
> +PR = "r8"
>
> inherit packagegroup
>
> -RDEPENDS_${PN} = "\
> +GST_010_DEPS = " \
> gstreamer-dev \
> gst-plugins-base-dev \
> gst-plugins-good-dev \
> gst-plugins-bad-dev \
> gst-ffmpeg-dev \
> "
> +
> +GST_1X_DEPS = " \
> + gstreamer1.0-dev \
> + gstreamer1.0-plugins-base-dev \
> + gstreamer1.0-plugins-good-dev \
> + gstreamer1.0-plugins-bad-dev \
> + gstreamer1.0-libav-dev \
> +"
> +
> +RDEPENDS_${PN} = " \
> + ${@base_conditional('GST_PROVIDER', '0.10', "${GST_010_DEPS}", "${GST_1X_DEPS}", d)} \
> +"
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> index d88e4c8..b449dfe 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> @@ -1,12 +1,12 @@
> DESCRIPTION = "Task to add gstreamer and gstreamer plugins"
> LICENSE = "MIT"
> -PR = "r11"
> +PR = "r12"
>
> inherit packagegroup
>
> PACKAGE_ARCH = "${MACHINE_ARCH}"
>
> -GSTREAMER = " \
> +GST_010_DEPS = " \
> gstreamer \
> gst-plugins-base \
> gst-plugins-good \
> @@ -14,14 +14,27 @@ GSTREAMER = " \
> gst-plugins-base-meta \
> gst-plugins-good-meta \
> gst-plugins-bad-meta \
> + gst-ffmpeg \
> +"
> +
> +GST_1X_DEPS = " \
> + gstreamer1.0 \
> + gstreamer1.0-plugins-base \
> + gstreamer1.0-plugins-good \
> + gstreamer1.0-plugins-bad \
> + gstreamer1.0-plugins-base-meta \
> + gstreamer1.0-plugins-good-meta \
> + gstreamer1.0-plugins-bad-meta \
> + gstreamer1.0-libav \
> "
>
> +GST_PLUGINS_PREFIX = "${@base_conditional('GST_PROVIDER', '0.10', 'gst-plugins', 'gstreamer1.0-plugins', d)}"
> +
> GSTREAMER_ACCEL_MM = ""
> -GSTREAMER_ACCEL_MM_omap-a15 = "${@base_contains('MACHINE_FEATURES', 'mmip', 'gst-plugins-ducati', '', d)}"
> -GSTREAMER_ACCEL_MM_append_dra7xx = "${@base_contains('MACHINE_FEATURES', 'mmip', ' gst-plugins-vpe', '', d)}"
> +GSTREAMER_ACCEL_MM_omap-a15 = "${@base_contains('MACHINE_FEATURES', 'mmip', "${GST_PLUGINS_PREFIX}-ducati", '', d)}"
> +GSTREAMER_ACCEL_MM_append_dra7xx = "${@base_contains('MACHINE_FEATURES', 'mmip', " ${GST_PLUGINS_PREFIX}-vpe", '', d)}"
>
> RDEPENDS_${PN} = "\
> - ${GSTREAMER} \
> + ${@base_conditional('GST_PROVIDER', '0.10', "${GST_010_DEPS}", "${GST_1X_DEPS}", d)} \
> ${GSTREAMER_ACCEL_MM} \
> - gst-ffmpeg \
> "
> diff --git a/meta-arago-extras/recipes-graphics/glsdk-example-apps/glsdk-example-apps.bb b/meta-arago-extras/recipes-graphics/glsdk-example-apps/glsdk-example-apps.bb
> index a6b45b8..9236d6d 100644
> --- a/meta-arago-extras/recipes-graphics/glsdk-example-apps/glsdk-example-apps.bb
> +++ b/meta-arago-extras/recipes-graphics/glsdk-example-apps/glsdk-example-apps.bb
> @@ -3,7 +3,7 @@ HOMEPAGE = "http://git.ti.com/glsdk"
> LICENSE = "BSD"
> LIC_FILES_CHKSUM = "file://COPYING;md5=10a9abb9c5bb19edd83a8cf66eef7148"
>
> -DEPENDS = "gstreamer libdrm ti-ipc"
> +DEPENDS = "${@base_conditional('GST_PROVIDER', '0.10', 'gstreamer', 'gstreamer1.0', d)} libdrm ti-ipc"
>
> COMPATIBLE_MACHINE = "dra7xx-evm"
>
> @@ -14,4 +14,4 @@ inherit autotools pkgconfig
> SRC_URI = "git://git.ti.com/glsdk/example-applications.git;protocol=git"
> SRCREV = "a21dd98d5686a6c4563f89966cb459158085dd1b"
>
> -PR = "r1"
> +PR = "r2"
next prev parent reply other threads:[~2015-04-27 18:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-27 17:53 [PATCH] gstreamer, conf: add GST_PROVIDER to switch between 0.10 and 1.x gstreamer Denys Dmytriyenko
2015-04-27 18:50 ` Jacob Stiffler [this message]
2015-04-27 19:07 ` Denys Dmytriyenko
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=553E84F4.5080403@ti.com \
--to=j-stiffler@ti.com \
--cc=denis@denix.org \
--cc=meta-arago@arago-project.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.