On Tue, Feb 10, 2015 at 11:43:29AM -0500, Nelson, Sam wrote:
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Tuesday, February 10, 2015 11:37 AM
> > To: Nelson, Sam
> > Cc: Cooper Jr., Franklin; [email protected]
> > Subject: Re: [meta-arago] [PATCH] packagegroup-arago-base-tisdk: Make
> > psplash conditional with splash feature
> > 
> > On Tue, Feb 10, 2015 at 04:30:57PM +0000, Nelson, Sam wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Cooper Jr., Franklin
> > > > Sent: Tuesday, February 10, 2015 11:16 AM
> > > > To: Nelson, Sam; [email protected]
> > > > Subject: RE: [meta-arago] [PATCH] packagegroup-arago-base-tisdk: Make
> > > > psplash conditional with splash feature
> > > >
> > > > Splash is not a distro feature we use so this will essentially remove 
> > > > it from
> > all
> > > > devices.
> > > >
> > > > You can either put this in a packagegroup that isn't included for 
> > > > headless
> > > > filesystems. But pick one that makes sense.
> > > > You can conditional include or not include psplash depending on 
> > > > machines.
> > > > You can use machine features and check for touchscreen. Although some
> > kind
> > > > of generic "display" machine feature would be better. But it currently
> > doesn't
> > > > exist.
> > > > Or you can add splash as a distro feature for the various branding files
> > > > excluding yours.
> > 
> > > [Sam] Do you see any issues using the "IMAGE_FEATURES" instead?
> > > Core-image-base include splash by default ( so all products will include
> > > psplash).
> > 
> > That's the wrong assumption.
> > 
> [Sam] Ok.  Thanks for the clarification. Is there any other alternative 
> suggestion?  

We don't use core-image-base.bb from oe-core, so we need to enable "splash" 
IMAGE_FEATURES in our image recipes. See the patch I just sent and provide 
feedback.


> > > And I am thinking of removing it for Keystone using branding file
> > > and can be done for any other headless distribution similarly.
> > 
> > Won't work due to above.
> > 
> > 
> > > > > -----Original Message-----
> > > > > From: [email protected] [mailto:meta-arago-
> > > > > [email protected]] On Behalf Of Nelson, Sam
> > > > > Sent: Tuesday, February 10, 2015 10:01 AM
> > > > > To: [email protected]
> > > > > Subject: [meta-arago] [PATCH] packagegroup-arago-base-tisdk: Make
> > psplash
> > > > > conditional with splash feature
> > > > >
> > > > > - This is meant to avoid error message with headless distribution
> > > > >
> > > > > Signed-off-by: Sam Nelson <[email protected]>
> > > > > ---
> > > > >  .../packagegroups/packagegroup-arago-base-tisdk.bb |    4 ++--
> > > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git 
> > > > > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > > > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > > > index 7652622..7b71aa4 100644
> > > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > > > base-tisdk.bb
> > > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > arago-
> > > > > base-tisdk.bb
> > > > > @@ -1,6 +1,6 @@
> > > > >  DESCRIPTION = "Additional packages beyond console packages shared by
> > TI
> > > > > SDKs"
> > > > >  LICENSE = "MIT"
> > > > > -PR = "r15"
> > > > > +PR = "r16"
> > > > >
> > > > >  inherit packagegroup
> > > > >
> > > > > @@ -12,7 +12,7 @@ RDEPENDS_${PN} = "\
> > > > >      libpcre \
> > > > >      iptables \
> > > > >      iperf \
> > > > > -    psplash \
> > > > > +    ${@base_contains('DISTRO_FEATURES', 'splash', 'psplash', '',d)} \
> > > > >      arago-gpl-notice \
> > > > >      arago-feed-config \
> > > > >      nfs-utils-client \
> > > > > --
> > > > > 1.7.9.5
> > > > >
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > [email protected]
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > _______________________________________________
> > > meta-arago mailing list
> > > [email protected]
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to