All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Ciprian Ciubotariu <cheepeero@gmx.net>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 00/10] Initial systemd integration
Date: Tue, 22 Jan 2013 22:29:24 +0000	[thread overview]
Message-ID: <1358893764.6356.28.camel@ted> (raw)
In-Reply-To: <2738897.RqdcWyXQgT@pink>

On Wed, 2013-01-23 at 00:04 +0200, Ciprian Ciubotariu wrote:
> On Monday 21 January 2013 12:12:14 Burton, Ross wrote:
> > On 21 January 2013 03:30, Ciprian Ciubotariu <cheepeero@gmx.net> wrote:
> > > However, with oe-core/meta providing a default embedded policy, higher
> > > layers need to remove sysvinit or systemd stuff from base recipes, which
> > > is
> > > against bitbake's additive language design (only append/prepend functions,
> > > no -= operator) and against separating concerns.
> > 
> > If you don't do a systemd build, you won't get any of the systemd
> > files.  Ditto, sysvinit (well, that's the goal - as it was an
> > assumption until now that needs work still).
> > 
> 
> Does that mean that I can disable the default init manager somehow, and 
> provide my own?

Its always been possible to provide your own init manager. The initramfs
filesystems do this, as does poky-tiny, you simply provide your own init
system dependencies and pull them in instead of sysvinit or systemd.

On more complex images, init systems are a little more complex than that
though.

> > An oe-core without *any* init system will be very cumbersome - every
> > service will need a bbappend to actually work, with the subsequent
> > maintenance costs.
> > 
> 
> I fail to see the overhead of maintaining a feature in one file, and the init-
> manager part of the same feature in another file. The actual complexity is the 
> same as when adding an use-flag like configuration variable in a single-file 
> recipe; perhaps one avoids hitting "Open..." on the editor.
> 
> However, if I understand correctly, one can disable the default OE policy for 
> an init manager, though not by choice of different layers, but via having 
> systemd or not in DISTRO_FEATURES. 
> 
> Does this means that 
> 
> - having DISTRO_FEATURES += "systemd" we get a system with systemd;
> - DISTRO_FEATURES += "sysvinit" makes a system with sysvinit, and 
> - leaving DISTRO_FEATURES blank leaves us with a system with no init manager, 
> to which we can add our own?

Those flags control the way some recipes build. Those recipes have
source code which does different things depending on whether sysvinit or
systemd is configured. This is not our code, its upstream. You can
control the way any package builds using the usual overrides mechanism.
The system is about as configurable and customisable as you can get.

> I guess the most important aspect I am trying to communicate is: please do not 
> provide any by default.

By default users expect the system to boot and to work. They don't
expect to have to add in layers just to get the system to initialise. We
need to support a sensible core offering of init features. We support
minimal configurations like tiny and the initramfs scripts, we now also
support a choice between systemd and clasic sysvinit. This doesn't mean
you have to use them, you're free to override and configure the system
as you wish, as you have always been able to do.

The request for no default simply does not make any sense. The fact we
have a default doesn't stop you adding your own (as the meta-systemd
layer has demonstrated).

Cheers,

Richard





  reply	other threads:[~2013-01-22 22:45 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-19 22:47 [PATCH 00/10] Initial systemd integration Ross Burton
2013-01-19 22:47 ` [PATCH 01/10] default-distrovars: Add DISTRO_FEATURES_INITMAN to DISTRO_FEATURES Ross Burton
2013-01-23 11:38   ` [PATCH] bitbake.conf: unbreak all builds with custom DISTRO_FEATURES Marcin Juszkiewicz
2013-01-23 11:43     ` Burton, Ross
2013-01-23 11:47       ` Marcin Juszkiewicz
2013-01-19 22:47 ` [PATCH 02/10] default-providers: Automatically set PREFERRED_PROVIDER_udev Ross Burton
2013-01-20 23:12   ` Martin Jansa
2013-01-20 23:15     ` Martin Jansa
2013-01-19 22:47 ` [PATCH 03/10] dbus: respect systemd distro feature Ross Burton
2013-01-19 22:47 ` [PATCH 04/10] systemd: add systemd recipes Ross Burton
2013-01-20 23:11   ` Martin Jansa
2013-01-21 12:07     ` Burton, Ross
2013-01-19 22:47 ` [PATCH 05/10] default-providers: Add systemd option to PREFERRED_PROVIDER_udev Ross Burton
2013-01-19 22:47 ` [PATCH 06/10] packagegroup-core-boot: install systemd-compat-units on systemd images Ross Burton
2013-01-19 22:47 ` [PATCH 07/10] update-rc.d: disable update-rc.d.bbclass when systemd enabled Ross Burton
2013-01-19 22:47 ` [PATCH 08/10] base-files: add fstab for systemd based systems Ross Burton
2013-01-19 22:47 ` [PATCH 09/10] packagegroup-core-boot: only install initscripts if we're using sysvinit Ross Burton
2013-01-19 22:47 ` [PATCH 10/10] libpam: register PAM session with logind Ross Burton
2013-01-20 18:34 ` [PATCH 00/10] Initial systemd integration Saul Wold
2013-01-20 20:21   ` Burton, Ross
2013-01-21  8:59     ` Martin Jansa
2013-01-21  9:21       ` Burton, Ross
2013-01-21  3:57   ` Saul Wold
2013-01-21  8:08     ` Burton, Ross
2013-01-21  8:19       ` Eric Bénard
2013-01-21  8:47         ` Radu Moisan
2013-01-21 10:14         ` Richard Purdie
2013-01-21 12:09           ` Burton, Ross
2013-01-21 12:10     ` Burton, Ross
2013-01-21  3:30 ` Ciprian Ciubotariu
2013-01-21 12:12   ` Burton, Ross
2013-01-21 16:57     ` Saul Wold
2013-01-21 17:00       ` Burton, Ross
2013-01-22 22:04     ` Ciprian Ciubotariu
2013-01-22 22:29       ` Richard Purdie [this message]
2013-01-22  9:30 ` ChenQi
2013-01-22 10:48   ` Radu Moisan

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=1358893764.6356.28.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=cheepeero@gmx.net \
    --cc=openembedded-core@lists.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.