* [PATCH] gstreamer: Fixed dependency problem with -meta packages
@ 2013-05-05 20:50 Carlos Rafael Giani
2013-05-07 9:55 ` Burton, Ross
0 siblings, 1 reply; 4+ messages in thread
From: Carlos Rafael Giani @ 2013-05-05 20:50 UTC (permalink / raw)
To: openembedded-core
Installing the gstreamer -meta packages was not possible, because the
-apps and -glib packages were missing. Allowing for empty -apps/glib
packages fixes this.
Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>
---
meta/recipes-multimedia/gstreamer/gst-plugins-package.inc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
index 7ae3ef6..9cab392 100644
--- a/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
+++ b/meta/recipes-multimedia/gstreamer/gst-plugins-package.inc
@@ -25,9 +25,12 @@ python populate_packages_prepend () {
}
# metapkg has runtime dependency on PN
-# each plugin depends on PN, plugin-dev on PN-dev and plugin-staticdev on PN-staticdev
+# each plugin depends on PN, plugin-dev on PN-dev and plugin-staticdev on PN-staticdev,
+# and plugin-meta on plugin-glib and plugin-apps
# so we need them even when empty (like in gst-plugins-good case)
ALLOW_EMPTY_${PN} = "1"
+ALLOW_EMPTY_${PN}-apps = "1"
+ALLOW_EMPTY_${PN}-glib = "1"
ALLOW_EMPTY_${PN}-dev = "1"
ALLOW_EMPTY_${PN}-staticdev = "1"
--
1.8.2.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] gstreamer: Fixed dependency problem with -meta packages
2013-05-05 20:50 [PATCH] gstreamer: Fixed dependency problem with -meta packages Carlos Rafael Giani
@ 2013-05-07 9:55 ` Burton, Ross
2013-05-07 10:29 ` Carlos Rafael Giani
0 siblings, 1 reply; 4+ messages in thread
From: Burton, Ross @ 2013-05-07 9:55 UTC (permalink / raw)
To: Carlos Rafael Giani; +Cc: openembedded-core
On 5 May 2013 21:50, Carlos Rafael Giani <dv@pseudoterminal.org> wrote:
> Installing the gstreamer -meta packages was not possible, because the
> -apps and -glib packages were missing. Allowing for empty -apps/glib
> packages fixes this.
From what I can tell there's only one -apps package
(gst-plugins-base-apps) and one -glib package (gst-plugins-bad-glib),
so why not add those extra dependencies in -base and -bad explicitly
instead of creating empty packages for no reason?
Ross
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] gstreamer: Fixed dependency problem with -meta packages
2013-05-07 9:55 ` Burton, Ross
@ 2013-05-07 10:29 ` Carlos Rafael Giani
2013-05-07 10:55 ` Burton, Ross
0 siblings, 1 reply; 4+ messages in thread
From: Carlos Rafael Giani @ 2013-05-07 10:29 UTC (permalink / raw)
To: Burton, Ross; +Cc: openembedded-core
On 05/07/2013 11:55 AM, Burton, Ross wrote:
> On 5 May 2013 21:50, Carlos Rafael Giani <dv@pseudoterminal.org> wrote:
>> Installing the gstreamer -meta packages was not possible, because the
>> -apps and -glib packages were missing. Allowing for empty -apps/glib
>> packages fixes this.
> From what I can tell there's only one -apps package
> (gst-plugins-base-apps) and one -glib package (gst-plugins-bad-glib),
> so why not add those extra dependencies in -base and -bad explicitly
> instead of creating empty packages for no reason?
>
> Ross
True, however I wonder if somebody relies on these packages being there.
Is it acceptable to break compatibility here?
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] gstreamer: Fixed dependency problem with -meta packages
2013-05-07 10:29 ` Carlos Rafael Giani
@ 2013-05-07 10:55 ` Burton, Ross
0 siblings, 0 replies; 4+ messages in thread
From: Burton, Ross @ 2013-05-07 10:55 UTC (permalink / raw)
To: Carlos Rafael Giani; +Cc: openembedded-core
On 7 May 2013 11:29, Carlos Rafael Giani <dv@pseudoterminal.org> wrote:
> True, however I wonder if somebody relies on these packages being there.
> Is it acceptable to break compatibility here?
They haven't been there for a while and 1.4 is shipping without them
so I'd endorse cleaning them up instead of having empty packages.
Ross
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-05-07 11:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-05 20:50 [PATCH] gstreamer: Fixed dependency problem with -meta packages Carlos Rafael Giani
2013-05-07 9:55 ` Burton, Ross
2013-05-07 10:29 ` Carlos Rafael Giani
2013-05-07 10:55 ` Burton, Ross
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox