All of lore.kernel.org
 help / color / mirror / Atom feed
From: Koen Kooi <k.kooi@student.utwente.nl>
To: openembedded-devel@openembedded.org
Subject: [RFC] removing DISTRO_* vars from task-boot
Date: Tue, 17 Feb 2009 13:26:34 +0100	[thread overview]
Message-ID: <gneadq$s6g$1@ger.gmane.org> (raw)

Hi,

Lately I have been wondering how to change the /dev manager on a per 
image base, since I want udev for most images and busybox-mdev for 
special-purpose stuff. The catch is that I want to keep using task-boot 
for both.

Currently we have:

DISTRO_DEV_MANAGER ?= "${@base_contains("MACHINE_FEATURES", "kernel26", 
  "udev","",d)} "
DISTRO_INIT_MANAGER ?= "sysvinit sysvinit-pidof"
DISTRO_INITSCRIPTS ?= "initscripts"
DISTRO_LOGIN_MANAGER ?= "tinylogin"

in task-boot.bb

There are various ways we could attack this:

1) The image.bbclass way:

   Put the following in image.bbclass:

   IMAGE_DEV_MANAGER ?= "${@base_contains("MACHINE_FEATURES", 
"kernel26",  "udev","",d)} "
   IMAGE_INIT_MANAGER ?= "sysvinit sysvinit-pidof"
   IMAGE_INITSCRIPTS ?= "initscripts"
   IMAGE_LOGIN_MANAGER ?= "tinylogin"

   Pros:
   * Takes immediate effect
   * Can have distro defaults, overridable per image

   Cons:
   * marks them as 'user installed'
   * you can't 'debootstrap' from tasks anymore, you need to encode 
knowledge about the above vars into the debootstrap script

2) Make task-boot nostamp

   Pros:
   * takes effect on image build time
   * Can have distro defaults, overridable per image

   Cons:
   * QA hell, there's no way to tell what's inside task-boot_1.0-r87.ipk 
from the outside

3) Have task-boot.bb create task-boot-mdev and task-boot-udev

   Pros:
   * clean
   * solves my immediate need

   Cons:
   * doesn't scale for other vars

4) Move the vars to RRECOMMENDS and filter them out with BAD_RECOMMENDATIONS

   Pros:
   * takes effect on image time
   * performs as expected

   Cons:
   * image build succeeds when no /dev manager is present

What are your opinions on all this?

regards,

Koen





             reply	other threads:[~2009-02-17 12:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-17 12:26 Koen Kooi [this message]
2009-02-17 16:35 ` [RFC] removing DISTRO_* vars from task-boot Tom Rini
2009-02-18 18:43   ` Koen Kooi
2009-02-18 19:22     ` Tom Rini
2009-02-19 10:40       ` Koen Kooi
2009-02-19 11:00         ` Sledz, Steffen
2009-02-19 11:26           ` Koen Kooi
2009-02-19 12:17             ` Sledz, Steffen
2009-02-19 11:14         ` Koen Kooi
2009-02-19 12:51           ` Marcin Juszkiewicz
2009-02-18 20:07     ` Philip Balister
2009-02-19 11:08 ` Marcin Juszkiewicz

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='gneadq$s6g$1@ger.gmane.org' \
    --to=k.kooi@student.utwente.nl \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=openembedded-devel@openembedded.org \
    /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.