On Fri, 2011-09-16 at 10:20 +0200, Koen Kooi wrote:
> some text here

It took all my restraint to not just reply with:
"""
NAK

<insert reason for NAK here, I can't be bothered to type it>
"""

We've been around in a few circles with this. The problem is that if we
apply this patch we have no clue which gst-plugin from the good, the bad
and the ugly provides something you're after to include in an image.
This results in bitbake being pretty clueless about whether a given
build will succeed or not. In general I'm not a fan of having
non-deterministic builds as they tend to annoy users.

If this position isn't acceptable then we'll probably have to move to a
situation where we list which plugins each of the packages builds and
drop the dyanmic provides. That is a maintenance pain and I don't take
that step lightly but I don't see any other options. I'm open to
suggestions though.

Cheers,

Richard

> Signed-off-by: Koen Kooi <k...@dominion.thruhere.net>
> ---
>  .../gstreamer/gst-plugins-package.inc              |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc 
> b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> index 796df14..dae9416 100644
> --- a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> @@ -6,8 +6,8 @@ python populate_packages_prepend () {
>       glibdir = bb.data.expand('${libdir}', d)
>  
>       do_split_packages(d, glibdir, '^lib(.*)\.so\.*', 'lib%s', 'gstreamer %s 
> library', extra_depends='', allow_links=True)
> -     do_split_packages(d, gst_libdir, 'libgst(.*)\.so$', 
> bb.data.expand('${PN}-%s', d), 'GStreamer plugin for %s', postinst=postinst, 
> extra_depends=bb.data.expand('${PN}',d))
> -     do_split_packages(d, gst_libdir, 'libgst(.*)\.l?a$', 
> bb.data.expand('${PN}-%s-dev', d), 'GStreamer plugin for %s (development 
> files)', extra_depends=bb.data.expand('${PN}-dev',d))
> +     do_split_packages(d, gst_libdir, '^libgst(.*)\.so$', 'gst-plugin-%s', 
> 'GStreamer plugin for %s', postinst=postinst, extra_depends='')
> +     do_split_packages(d, gst_libdir, '^libgst(.*)\.l?a$', 
> 'gst-plugin-%s-dev', 'GStreamer plugin for %s (development files)', 
> extra_depends='')
>       
>       pn = bb.data.getVar('PN', d, 1)
>       metapkg =  pn + '-meta'



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

Reply via email to