All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Zanussi <tom.zanussi@intel.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/2] gst-ffmpeg: add LICENSE_FLAGS
Date: Fri, 14 Sep 2012 08:34:02 -0500	[thread overview]
Message-ID: <1347629642.19399.124.camel@empanada> (raw)
In-Reply-To: <CAJTo0LYHQjL=2vSP=SNcT9csSd8vyGm=d-9_8TRh5SC18h7bCg@mail.gmail.com>

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





  reply	other threads:[~2012-09-14 13:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2012-09-13 11:03 ` [PATCH 2/2] gst-fluendo-mpegdemux: " Paul Eggleton

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=1347629642.19399.124.camel@empanada \
    --to=tom.zanussi@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.intel.com \
    --cc=ross.burton@intel.com \
    /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.