Re: [for master PATCH] gstreamer: Add missing FILES_PN to populate rootfs

From: Denys Dmytriyenko <denys@ti.com>
To: Karthik Ramanan <a0393906@ti.com>
Cc: meta-arago@arago-project.org
Subject: Re: [for master PATCH] gstreamer: Add missing FILES_PN to populate rootfs
Date: Tue, 2 Dec 2014 20:45:25 -0500	[thread overview]
Message-ID: <20141203014525.GI5113@edge> (raw)
In-Reply-To: <1416990689-13265-1-git-send-email-a0393906@ti.com>

On Wed, Nov 26, 2014 at 02:01:28PM +0530, Karthik Ramanan wrote:
> All the libraries and binaries from GST, base-plugins,
> good-plugins and bad-plugins need to be populated into
> the target filesystem.

NAK - this is not the correct way to do what you explain here!

The code I've added to this .inc file splits out all the libraries and plugins 
into individual packages, similar to how it's done in the upstream gstreamer 
recipes:

http://arago-project.org/git/?p=meta-arago.git;a=commitdiff;h=75b485f1208f605dd7da70bf61b0a078c9c863a8

You need to list necessary library packages in the corresponding packagegroup 
to be added to the target filesystem. Or, alternatively, you can pull a "meta" 
package that lists all produced libraries and plugins. packagegroup-arago-gst 
does that already - it includes all gst-plugins-*-meta and that should already 
be set to point to the corresponding "mm-accel" packages with necessary 
PREFERRED_PROVIDER statements in arago-prefs.inc

Moreover, I don't see any files not being packaged, which would need to be 
picked up by your below FILES_* statement. The only warning I see from bitbake 
as related to gstreamer, are:

WARNING: QA Issue: gst-plugins-ducati: Files/directories were installed but not shipped
  /usr/lib/gstreamer-0.10/.debug
  /usr/lib/gstreamer-0.10/.debug/libgstducati.so
WARNING: QA Issue: gst-plugins-vpe: Files/directories were installed but not shipped
  /usr/lib/gstreamer-0.10/.debug
  /usr/lib/gstreamer-0.10/.debug/libgstvpe.so
  /usr/lib/gstreamer-0.10/.debug/libgstperf.so

So, the question would be - what are you trying to fix here?

-- 
Denys

> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
> ---
>  .../gstreamer/gst-plugins-package-mm-accel.inc     |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta-arago-extras/recipes-multimedia/gstreamer/gst-plugins-package-mm-accel.inc b/meta-arago-extras/recipes-multimedia/gstreamer/gst-plugins-package-mm-accel.inc
> index a1c81f3..68f643c 100644
> --- a/meta-arago-extras/recipes-multimedia/gstreamer/gst-plugins-package-mm-accel.inc
> +++ b/meta-arago-extras/recipes-multimedia/gstreamer/gst-plugins-package-mm-accel.inc
> @@ -15,3 +15,5 @@ python split_gstreamer10_packages () {
>      do_split_packages(d, gst_libdir, 'libgst(.*)\.la$', d.expand('${PN}-%s-dev'), 'GStreamer plugin for %s (development files)', extra_depends=d.expand('${PN}-dev'))
>      do_split_packages(d, gst_libdir, 'libgst(.*)\.a$', d.expand('${PN}-%s-staticdev'), 'GStreamer plugin for %s (static development files)', extra_depends=d.expand('${PN}-staticdev'))
>  }
> +
> +FILES_${PN} += "${libdir}/gstreamer-0.10/*"
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago