Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Saul Wold <sgw@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 02/28] classes/core-image: remove *_IMAGE_FEATURES variables
Date: Tue, 04 Sep 2012 09:30:28 +0100	[thread overview]
Message-ID: <1937737.ltCaXIobqa@helios> (raw)
In-Reply-To: <5045687E.7010505@linux.intel.com>

On Monday 03 September 2012 19:33:34 Saul Wold wrote:
> > -IMAGE_FEATURES += "apps-console-core ${SATO_IMAGE_FEATURES}"
> > +IMAGE_FEATURES += "apps-console-core package-management x11-base
> > apps-x11-core apps-x11-games x11-sato ssh-server-dropbear"
>
> Should package-management be here or should it be a DISTRO_FEATURE and
> have the ability to be enabled or disabled more on DISTRO basis?  I know
> that we explicitly disable it on core-image-minimal and have a
> ROOTFS_POSTPRESSES_COMMAND there, but I wonder about setting that up on
> a DISTRO basis?

DISTRO_FEATURES are usually items that influence changes at compilation time. I 
think it's also reasonable to expect people want to be able to produce images 
that have package management for development/testing/debugging and those that 
don't for production, side-by-side, in which case it has to be in 
IMAGE_FEATURES (especially as changing DISTRO_FEATURES on the fly will result 
in a lot of recompilation).

> It's part of the following:
> 
> meta/recipes-core/images/build-appliance-image.bb
> meta/recipes-graphics/images/core-image-clutter.bb
> meta/recipes-graphics/images/core-image-x11.bb
> meta/recipes-sato/images/core-image-sato.bb
> 
> What about LSB? core-image-base or core-image-basic?

The latter images never had package management enabled; I think core-image-
basic at least could reasonably have package management enabled though since 
it's meant to produce a more fully-featured Linux system (mental note, still 
need to rename some of those images). AIUI, LSB doesn't require full package 
management to be enabled, only that rpm packages are able to be installed; 
installing rpm should satisfy this. We would only want to enable full package 
management in the LSB images if it was in some way helpful to run the LSB 
tests.

I'm not entirely sure how we ended up with package management in the Sato 
images and not with some of the others - perhaps there was at one time a 
package management application within Sato or plans for one?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



  reply	other threads:[~2012-09-04  8:42 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-03 10:30 [PATCH 00/28] Package group fixes Paul Eggleton
2012-09-03 10:30 ` [PATCH 01/28] classes/core-image: remove SSHSERVER_IMAGE_FEATURES Paul Eggleton
2012-09-03 10:30 ` [PATCH 02/28] classes/core-image: remove *_IMAGE_FEATURES variables Paul Eggleton
2012-09-03 13:14   ` Paul Eggleton
2012-09-04  2:33   ` Saul Wold
2012-09-04  8:30     ` Paul Eggleton [this message]
2012-09-03 10:30 ` [PATCH 03/28] classes/core-image: remove erroneous nfs-server comment Paul Eggleton
2012-09-03 10:30 ` [PATCH 04/28] Rename task to packagegroup Paul Eggleton
2012-09-03 10:30 ` [PATCH 05/28] packagegroup-base: remove some useless items Paul Eggleton
2012-09-03 10:30 ` [PATCH 06/28] classes/packagegroup: Use package group nomenclature Paul Eggleton
2012-09-03 10:30 ` [PATCH 07/28] packagegroup-core-nfs: remove broken glibc-utils reference Paul Eggleton
2012-09-03 10:30 ` [PATCH 08/28] classes/packagegroup: add ability to disable complementary packages Paul Eggleton
2012-09-03 13:25   ` Paul Eggleton
2012-09-03 10:30 ` [PATCH 09/28] packagegroup-*: change to inherit from packagegroup.bbclass Paul Eggleton
2012-09-03 10:30 ` [PATCH 10/28] packagegroup-core-x11: move out Sato applications Paul Eggleton
2012-09-03 10:30 ` [PATCH 11/28] core-image-core: rename to core-image-x11 Paul Eggleton
2012-09-03 10:30 ` [PATCH 12/28] packagegroup-core-x11*: adjust X11 package groups Paul Eggleton
2012-09-03 10:30 ` [PATCH 13/28] packagegroup-qt4e: tidy up and rename Paul Eggleton
2012-09-03 10:30 ` [PATCH 14/28] packagegroup-core-gtk-directfb: remove superfluous -base package Paul Eggleton
2012-09-03 10:30 ` [PATCH 15/28] packagegroup-base: remove openswan from packagegroup-base-ipsec Paul Eggleton
2012-09-03 10:35   ` Phil Blundell
2012-09-03 10:37     ` Paul Eggleton
2012-09-03 10:50       ` Phil Blundell
2012-09-03 10:54         ` Paul Eggleton
2012-09-03 16:25           ` Koen Kooi
2012-09-03 16:34             ` Paul Eggleton
2012-09-04 10:33           ` Phil Blundell
2012-09-03 10:30 ` [PATCH 16/28] packagegroup-core-gtk-directfb: remove dropbear Paul Eggleton
2012-09-03 10:30 ` [PATCH 17/28] classes/packagegroup: disable superfluous tasks Paul Eggleton
2012-09-03 10:30 ` [PATCH 18/28] packagegroup-*: drop LIC_FILES_CHKSUM Paul Eggleton
2012-09-03 10:30 ` [PATCH 19/28] packagegroup-core-clutter: remove empty -apps/-tests tasks Paul Eggleton
2012-09-03 10:30 ` [PATCH 20/28] packagegroup-core-qt: rename -demos to -demoapps Paul Eggleton
2012-09-03 10:30 ` [PATCH 21/28] packagegroup-core-console: remove Paul Eggleton
2012-09-03 10:30 ` [PATCH 22/28] connman: add runtime dependency on dbus Paul Eggleton
2012-09-03 10:30 ` [PATCH 23/28] ofono: " Paul Eggleton
2012-09-03 10:30 ` [PATCH 24/28] packagegroup-*: set reasonable SUMMARY/DESCRIPTION Paul Eggleton
2012-09-03 10:30 ` [PATCH 25/28] packagegroup-core-basic: tidy up Paul Eggleton
2012-09-03 10:30 ` [PATCH 26/28] packagegroup-core-lsb: rework Paul Eggleton
2012-09-03 10:30 ` [PATCH 27/28] core-image-basic: tidy up Paul Eggleton
2012-09-03 10:30 ` [PATCH 28/28] packagegroup-*: add RPROVIDES for backwards compatibility Paul Eggleton
2012-09-03 10:39 ` [PATCH 00/28] Package group fixes Paul Eggleton
2012-09-03 16:26 ` Koen Kooi
2012-09-03 16:35   ` Paul Eggleton
2012-09-04  8:38     ` Koen Kooi
2012-09-04  8:45       ` Paul Eggleton
2012-09-04  9:46 ` Paul Eggleton
2012-09-04 12:36 ` Richard Purdie
2012-09-04 12:58 ` Martin Jansa
2012-09-04 13:09   ` Paul Eggleton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1937737.ltCaXIobqa@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=sgw@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox