On Fri, 6 Nov 2020 at 22:45, Tammana, Gowtham <[email protected]> wrote:
>
>
>
> >-----Original Message-----
> >From: Paul Barker <[email protected]>
> >Sent: Friday, November 06, 2020 3:06 AM
> >To: Tammana, Gowtham <[email protected]>
> >Cc: [email protected]
> >Subject: Re: [EXTERNAL] [meta-arago] [master/dunfell PATCH] matrix-gui-pm-
> >demos: Allow empty package
> >
> >On Thu, 5 Nov 2020 at 22:54, Tammana, Gowtham <[email protected]> wrote:
> >>
> >> >-----Original Message-----
> >> >From: [email protected] <meta-arago-bounces@arago-
> >> >project.org> On Behalf Of Paul Barker
> >> >Sent: Thursday, November 05, 2020 4:38 AM
> >> >To: [email protected]
> >> >Subject: [EXTERNAL] [meta-arago] [master/dunfell PATCH] matrix-gui-pm-
> >> >demos: Allow empty package
> >> >
> >> >The matrix-gui-pm-demos recipe splits its binaries into several
> >> >subpackages leaving the default package empty. The dependencies for the
> >> >default package are set so it makes sense to allow this empty package
> >to
> >> >be created.
> >> >
> >> >Signed-off-by: Paul Barker <[email protected]>
> >> >---
> >> > meta-arago-extras/recipes-core/matrix/matrix-gui-pm-demos_2.0.bb | 1 +
> >> > 1 file changed, 1 insertion(+)
> >> >
> >> >diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-pm-
> >> >demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-pm-
> >> >demos_2.0.bb
> >> >index 0116f5ea..a9ebb84f 100644
> >> >--- a/meta-arago-extras/recipes-core/matrix/matrix-gui-pm-demos_2.0.bb
> >> >+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-pm-demos_2.0.bb
> >> >@@ -10,6 +10,7 @@ S = "${WORKDIR}/git/power_apps"
> >> >
> >> > # Make sure power submenu and app images has been installed
> >> > RDEPENDS_${PN} += "matrix-gui-apps-images matrix-gui-submenus-power
> >> >matrix-gui-generic-pm"
> >> >+ALLOW_EMPTY_${PN} = "1"
> >>
> >> Just curious, I assume that the deployment of the dependent packages is
> >still
> >> ensured even without this change. This just makes it apparent to the
> >user that
> >> the default package is empty. Or is it that the deployment of dependent
> >packages
> >> is broken without this.
> >
> >Without this change the matrix-gui-pm-demos package is not created and
> >cannot be installed.
>
> Yes, but as the package here is empty trying to understand why it needs to
> be installed in the first place.
>
> Now that the package is created, wondering if the dependencies with the
> sub-packages needs to be listed as well something like
>
> RDEPENDS_${PN} += ${PN}-<subpackages} or
>
> RDEPENDS_${PN}-<subpackages> = ${PN}

I'm not sure I understand your comment here. RDEPENDS_${PN} is set on
the line above my change. So effectively ${PN} is a packagegroup which
will install the listed dependencies. But without ALLOW_EMPTY being
set it won't be created.

The other thing to consider is that ${PN} is a default dependency of
${PN}-dev and so not creating a ${PN} package can result in image
features like dev-pkgs breaking.

-- 
Paul Barker
Konsulko Group
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to