All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: "Robert P. J. Day" <rpjday@crashcourse.ca>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: what is "packagegroup-core-nfs-server"?
Date: Wed, 28 Nov 2012 13:37 +0000	[thread overview]
Message-ID: <5890106.Vb1bZqkAH2@helios> (raw)
In-Reply-To: <alpine.DEB.2.02.1211280808130.578@oneiric>

On Wednesday 28 November 2012 08:14:44 Robert P. J. Day wrote:
> On Wed, 28 Nov 2012, Martin Jansa wrote:
> > On Wed, Nov 28, 2012 at 07:58:45AM -0500, Robert P. J. Day wrote:
> > > On Tue, 27 Nov 2012, Saul Wold wrote:
> > > > On 11/27/2012 08:18 PM, Robert P. J. Day wrote:
> > > > >    poking around core-image.bbclass, and noticed this:
> > > > > PACKAGE_GROUP_nfs-server = "packagegroup-core-nfs-server"
> > > > > 
> > > > > except i don't see the corresponding packagegroup-core-nfs-server.bb
> > > > > recipe file.  there *is* a packagegroup-core-nfs.bb file; how is the
> > > > > above processed?
> > > > 
> > > > It's defined in recipes-core/packagegroups/packagegroup-core-nfs.bb
> > > > 
> > > > The line
> > > > PACKAGES = "${PN}-server"
> > > > provides the key you are looking for.
> > > > 
> > >   a followup, for which i'm quite prepared to embarrass myself --
> > > 
> > > where is the "splash" package group defined?  i can see this in
> > > image.bbclass:
> > > 
> > > SPLASH ?= "psplash"
> > > PACKAGE_GROUP_splash = "${SPLASH}"
> > > 
> > > and numerous images add that "splash" package group.  but my first
> > > impression was that any package would be defined under some
> > > "packagegroups/" directory, and would necessarily need to "inherit
> > > packagegroup".
> > > 
> > >   so what am i missing?
> > 
> > PACKAGE_GROUP_foo can be provided by any recipe, not only
> > packagegroup-*
> 
>   ok, that's useful information that doesn't seem to be documented
> anywhere (or is it?).
> 
>   however, if a regular recipe can be used as the basis for a package
> group, would it not still have to contain "inherit packagegroup" to be
> defined as a package group?  or can any regular recipe act as its own
> package group?  i examined the "psplash" recipe and i saw nothing that
> suggested it was available as a package group.
> 
>   i realize these are nitpicky questions but it's the sort of thing
> someone's guaranteed to ask me at some point, so i want to nail it
> down.

So the bit that might be missing here is that the PACKAGE_GROUP structure and 
the packagegroup recipes/class are actually not really related, except that 
the latter may be used to populate the former. "packagegroup" is the new name 
for what used to be known as a "task" in versions before danny and much better 
reflects what these recipes do. PACKAGE_GROUP_* actually came first as a way to 
define items for IMAGE_FEATURES that map to one or more packages, and I'm not 
sure the name choice was a particularly good one - I wonder if it would be 
worth considering renaming it to something like IMAGE_FEATURE_PACKAGES in 
order to avoid confusion, although obviously every rename like this has a cost 
associated.

In any case, I would definitely recommend using the term "package group" to 
refer to packagegroup recipes only, otherwise you're only likely to increase 
people's level of confusion.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



  reply	other threads:[~2012-11-28 13:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-28  4:18 what is "packagegroup-core-nfs-server"? Robert P. J. Day
2012-11-28  4:27 ` Saul Wold
2012-11-28  6:18   ` Robert P. J. Day
2012-11-28 12:58   ` Robert P. J. Day
2012-11-28 13:01     ` Martin Jansa
2012-11-28 13:14       ` Robert P. J. Day
2012-11-28 13:37         ` Paul Eggleton [this message]
2012-11-28 13:45           ` Robert P. J. Day
2012-11-28 14:02             ` Paul Eggleton
2012-11-28 13:55           ` Robert P. J. Day
2012-11-28 14:08             ` 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=5890106.Vb1bZqkAH2@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=rpjday@crashcourse.ca \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.