Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/2] A couple of minor fixes
@ 2012-09-13 11:03 Paul Eggleton
  2012-09-13 11:03 ` [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS Paul Eggleton
  2012-09-13 11:03 ` [PATCH 2/2] gst-fluendo-mpegdemux: " Paul Eggleton
  0 siblings, 2 replies; 6+ messages in thread
From: Paul Eggleton @ 2012-09-13 11:03 UTC (permalink / raw)
  To: openembedded-core

The following changes since commit 835654994574c158d6324218ebe000bd2ef9a792:

  rt: Add hwlatdetect to rt images (2012-09-12 15:11:12 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/fixes1
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/fixes1

Paul Eggleton (2):
  gst-ffmpeg: add LICENSE_FLAGS
  gst-fluendo-mpegdemux: add LICENSE_FLAGS

 .../gstreamer/gst-ffmpeg_0.10.13.bb                |    1 +
 .../gstreamer/gst-fluendo-mpegdemux_0.10.71.bb     |    1 +
 2 files changed, 2 insertions(+)

-- 
1.7.9.5




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS
  2012-09-13 11:03 [PATCH 0/2] A couple of minor fixes Paul Eggleton
@ 2012-09-13 11:03 ` Paul Eggleton
  2012-09-14  6:28   ` Saul Wold
  2012-09-13 11:03 ` [PATCH 2/2] gst-fluendo-mpegdemux: " Paul Eggleton
  1 sibling, 1 reply; 6+ messages in thread
From: Paul Eggleton @ 2012-09-13 11:03 UTC (permalink / raw)
  To: openembedded-core

This likely requires some form of license to use in a commercial
product.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 .../gstreamer/gst-ffmpeg_0.10.13.bb                |    1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
index efab2cf..2107704 100644
--- a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
@@ -10,6 +10,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
                     file://gst-libs/ext/libav/COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
                     file://gst-libs/ext/libav/COPYING.LGPLv2.1;md5=e344c8fa836c3a41c4cbd79d7bd3a379 \
                     file://gst-libs/ext/libav/COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
+LICENSE_FLAGS = "commercial"
 HOMEPAGE = "http://www.gstreamer.net/"
 DEPENDS = "gstreamer gst-plugins-base zlib bzip2"
 
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] gst-fluendo-mpegdemux: add LICENSE_FLAGS
  2012-09-13 11:03 [PATCH 0/2] A couple of minor fixes Paul Eggleton
  2012-09-13 11:03 ` [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS Paul Eggleton
@ 2012-09-13 11:03 ` Paul Eggleton
  1 sibling, 0 replies; 6+ messages in thread
From: Paul Eggleton @ 2012-09-13 11:03 UTC (permalink / raw)
  To: openembedded-core

This likely requires some form of license to use in a commercial
product.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 .../gstreamer/gst-fluendo-mpegdemux_0.10.71.bb     |    1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/gstreamer/gst-fluendo-mpegdemux_0.10.71.bb b/meta/recipes-multimedia/gstreamer/gst-fluendo-mpegdemux_0.10.71.bb
index b939eff..580b9e8 100644
--- a/meta/recipes-multimedia/gstreamer/gst-fluendo-mpegdemux_0.10.71.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-fluendo-mpegdemux_0.10.71.bb
@@ -4,6 +4,7 @@ DESCRIPTION = "Fluendo closed-format mpeg video GStreamer plug-in"
 LICENSE = "MPLv1.1"
 LIC_FILES_CHKSUM = "file://COPYING;md5=be282f1c3cc9a98cc0dc5c2b25dfc510 \
                     file://src/gstmpegdemux.h;beginline=1;endline=19;md5=a9e90033f59897b91664d9f2a2ff01dd"
+LICENSE_FLAGS = "commercial"
 
 acpaths = "-I ${S}/common/m4 -I ${S}/m4"
 
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS
  2012-09-13 11:03 ` [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS Paul Eggleton
@ 2012-09-14  6:28   ` Saul Wold
  2012-09-14  9:23     ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2012-09-14  6:28 UTC (permalink / raw)
  To: Paul Eggleton; +Cc: openembedded-core

On 09/13/2012 04:03 AM, Paul Eggleton wrote:
> This likely requires some form of license to use in a commercial
> product.
>
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
>   .../gstreamer/gst-ffmpeg_0.10.13.bb                |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
> index efab2cf..2107704 100644
> --- a/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
> +++ b/meta/recipes-multimedia/gstreamer/gst-ffmpeg_0.10.13.bb
> @@ -10,6 +10,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>                       file://gst-libs/ext/libav/COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
>                       file://gst-libs/ext/libav/COPYING.LGPLv2.1;md5=e344c8fa836c3a41c4cbd79d7bd3a379 \
>                       file://gst-libs/ext/libav/COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
> +LICENSE_FLAGS = "commercial"
>   HOMEPAGE = "http://www.gstreamer.net/"
>   DEPENDS = "gstreamer gst-plugins-base zlib bzip2"
>
>

Failures from MUT:


http://autobuilder.yoctoproject.org:8010/builders/meta-intel-gpl/builds/47/steps/shell_42/logs/stdio

Seems to add some new ERRORS:

ERROR: Nothing PROVIDES 'libcroco-native' (but 
virtual:native:/srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/meta/recipes-gnome/librsvg/librsvg_2.32.1.bb 
DEPENDS on or otherwise requires it)
ERROR: Nothing RPROVIDES 'gst-ffmpeg' (but 
/srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/yocto/meta-intel/common/recipes-multimedia/gstreamer/gst-va-intel.bb 
RDEPENDS on or otherwise requires it)
ERROR: gst-ffmpeg was skipped: because it has a restricted license not 
whitelisted in LICENSE_FLAGS_WHITELIST
NOTE: Runtime target 'gst-ffmpeg' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['gst-ffmpeg']
ERROR: Nothing RPROVIDES 'gst-va-intel-general' (but 
/srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/yocto/meta-intel/common/recipes-multimedia/gstreamer/gst-va-intel.bb 
RDEPENDS on or otherwise requires it)
ERROR: No eligible RPROVIDERs exist for 'gst-va-intel-general'
NOTE: Runtime target 'gst-va-intel-general' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['gst-va-intel-general']
ERROR: Nothing RPROVIDES 'gst-va-intel-video' (but 
/srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/yocto/meta-intel/common/recipes-multimedia/gstreamer/gst-va-intel.bb 
RDEPENDS on or otherwise requires it)
ERROR: No eligible RPROVIDERs exist for 'gst-va-intel-video'
NOTE: Runtime target 'gst-va-intel-video' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['gst-va-intel-video']



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS
  2012-09-14  6:28   ` Saul Wold
@ 2012-09-14  9:23     ` Burton, Ross
  2012-09-14 13:34       ` Tom Zanussi
  0 siblings, 1 reply; 6+ messages in thread
From: Burton, Ross @ 2012-09-14  9:23 UTC (permalink / raw)
  To: Saul Wold; +Cc: Paul Eggleton, openembedded-core

On 14 September 2012 07:28, Saul Wold <sgw@linux.intel.com> wrote:
> ERROR: Nothing RPROVIDES 'gst-ffmpeg' (but
> /srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/yocto/meta-intel/common/recipes-multimedia/gstreamer/gst-va-intel.bb
> RDEPENDS on or otherwise requires it)

That's interesting, I wasn't aware gst-va required ffmpeg.  At the
most it should be a recommends as elements are pluggable, and to be
honest I'm pretty sure that's a bad dependency.

Ross



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS
  2012-09-14  9:23     ` Burton, Ross
@ 2012-09-14 13:34       ` Tom Zanussi
  0 siblings, 0 replies; 6+ messages in thread
From: Tom Zanussi @ 2012-09-14 13:34 UTC (permalink / raw)
  To: Burton, Ross; +Cc: Paul Eggleton, openembedded-core

On Fri, 2012-09-14 at 10:23 +0100, Burton, Ross wrote:
> On 14 September 2012 07:28, Saul Wold <sgw@linux.intel.com> wrote:
> > ERROR: Nothing RPROVIDES 'gst-ffmpeg' (but
> > /srv/home/pokybuild/yocto-autobuilder/yocto-slave/meta-intel-gpl/build/yocto/meta-intel/common/recipes-multimedia/gstreamer/gst-va-intel.bb
> > RDEPENDS on or otherwise requires it)
> 
> That's interesting, I wasn't aware gst-va required ffmpeg.  At the
> most it should be a recommends as elements are pluggable, and to be
> honest I'm pretty sure that's a bad dependency.
> 

So some of the meta-intel BSPs add this as RRECOMMENDS to get video
acceleration support under gstreamer:

MACHINE_EXTRA_RRECOMMENDS += "gst-va-intel"

and that adds a runtime dependency on gst-ffmpeg, which in turn uses a
local ffmpeg that does this to enable video video acceleration:

        --enable-vaapi \

but also does this to avoid being 'commercial':

        --enable-gpl \

So unless there's something that requires gst-ffmpeg to be commercial, I
don't think gst-ffmpeg actually needs commercial LICENSE_FLAGS.  What
was the reason it was added now after all this time in the first place?

Tom

> Ross
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-09-14 13:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-13 11:03 [PATCH 0/2] A couple of minor fixes Paul Eggleton
2012-09-13 11:03 ` [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS Paul Eggleton
2012-09-14  6:28   ` Saul Wold
2012-09-14  9:23     ` Burton, Ross
2012-09-14 13:34       ` Tom Zanussi
2012-09-13 11:03 ` [PATCH 2/2] gst-fluendo-mpegdemux: " Paul Eggleton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox