Patchwork gst-plugins-package: set ALLOW_EMPTY_${PN}

login
register
mail settings
Submitter Enrico Scholz
Date March 12, 2013, 12:20 p.m.
Message ID <1363090807-23944-1-git-send-email-enrico.scholz@sigma-chemnitz.de>
Download mbox | patch
Permalink /patch/46049/
State New
Headers show

Comments

Enrico Scholz - March 12, 2013, 12:20 p.m.
Every subpackage has a dependency on '${PN}' because
populate_packages_prepend() specifies "extra_depends=d.expand('${PN}')"
when splitting the subpackages.

Hence, '${PN}' must be always created

Although creation of -plugin-dev and -plugin-staticdev is flawed imo
(.la files are not for development purposes but useless resp. required
for module loading on non GNU/linux platforms, and .a files are not
created for plugins), patch sets ALLOW_EMPTY for -dev and -staticdev
packages for consistency reasons.

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
---
 meta/recipes-multimedia/gstreamer/gst-plugins-package.inc | 4 ++++
 1 file changed, 4 insertions(+)
Martin Jansa - March 12, 2013, 1:13 p.m.
On Tue, Mar 12, 2013 at 01:20:07PM +0100, Enrico Scholz wrote:
> Every subpackage has a dependency on '${PN}' because
> populate_packages_prepend() specifies "extra_depends=d.expand('${PN}')"
> when splitting the subpackages.
> 
> Hence, '${PN}' must be always created
> 
> Although creation of -plugin-dev and -plugin-staticdev is flawed imo
> (.la files are not for development purposes but useless resp. required
> for module loading on non GNU/linux platforms, and .a files are not
> created for plugins), patch sets ALLOW_EMPTY for -dev and -staticdev
> packages for consistency reasons.

I've sent the same a week ago:
http://lists.linuxtogo.org/pipermail/openembedded-core/2013-March/036752.html

> 
> Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
> ---
>  meta/recipes-multimedia/gstreamer/gst-plugins-package.inc | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> index 3e9ef92..2f2be34 100644
> --- a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
> @@ -24,6 +24,10 @@ python populate_packages_prepend () {
>      d.setVar('DESCRIPTION_' + metapkg, pn + ' meta package')
>  }
>  
> +ALLOW_EMPTY_${PN} = "1"
> +ALLOW_EMPTY_${PN}-dev = "1"
> +ALLOW_EMPTY_${PN}-staticdev = "1"
> +
>  PACKAGES += "${PN}-apps ${PN}-meta ${PN}-glib"
>  FILES_${PN}-apps = "${bindir}"
>  
> -- 
> 1.8.1.2
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Patch

diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
index 3e9ef92..2f2be34 100644
--- a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
+++ b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
@@ -24,6 +24,10 @@  python populate_packages_prepend () {
     d.setVar('DESCRIPTION_' + metapkg, pn + ' meta package')
 }
 
+ALLOW_EMPTY_${PN} = "1"
+ALLOW_EMPTY_${PN}-dev = "1"
+ALLOW_EMPTY_${PN}-staticdev = "1"
+
 PACKAGES += "${PN}-apps ${PN}-meta ${PN}-glib"
 FILES_${PN}-apps = "${bindir}"