From: Ciprian Ciubotariu <cheepeero@gmx.net>
To: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 00/10] Initial systemd integration
Date: Mon, 21 Jan 2013 05:30:08 +0200 [thread overview]
Message-ID: <1358739008.2945.29.camel@jethro.cheepee.homedns.org> (raw)
In-Reply-To: <cover.1358635193.git.ross.burton@intel.com>
[-- Attachment #1: Type: text/plain, Size: 12397 bytes --]
Hello.
I am merely a user of the free OpenEmbedded, and I was quite happy to
see that OpenEmbedded was heading towards providing multiple init
managers (be it sysvinit, systemd, openrc, upstart or what not).
However, from this series of patches it seems to me that systemd usage
is added to the openembedded-core meta layer, instead of a separate
meta-systemd layer.
As a result, I have to ask - why would the openembedded-core meta layer
provide a default init manager, instead of separating this choice in
other meta layers, like it was within meta-openembedded?
I find the following organization more orthogonal:
* meta - same as before
* meta-sysvinit - adds sysvinit recipes for meta packages
* meta-systemd - same for systemd-based systems
This would enable users (and yourself) to select whichever they want.
Also it would clean up base recipes from the core meta layer.
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.
Cipi
On Sat, 2013-01-19 at 22:47 +0000, Ross Burton wrote:
> Hi,
>
> This series has the goal of enabling core-image-minimal to boot using systemd.
> As such it touches no real services, just the infrastructural tweaks to make the
> init manager a distribution choice, systemd itself, and a few low-level changes.
>
> 95% of the credit obviously must go to the contributors of meta-systemd:
>
> Andreas Müller <schnitzeltony@googlemail.com>
> Denis 'GNUtoo' Carikli <GNUtoo@no-log.org>
> Holger Hans Peter Freyther <holger@moiji-mobile.com>
> Khem Raj <raj.khem@gmail.com>
> Koen Kooi <koen@dominion.thruhere.net>
> Martin Jansa <Martin.Jansa@gmail.com>
>
> And thanks to Radu Moisan <radu.moisan@intel.com> for the work pulling together
> a series for merging into oe-core.
>
> Confirmed that this series using sysvinit results in a zero-delta in
> core-image-minimal.
>
> Cheers,
> Ross
>
> The following changes since commit 349e5bea53b02cabf836b8943541a002ce9ba682:
>
> guilt: add git 1.8.x support (2013-01-18 13:28:10 +0000)
>
> are available in the git repository at:
>
> git://git.yoctoproject.org/poky-contrib ross/systemd-minimal
>
> for you to fetch changes up to 2a93db0fd8302290ac0f2c6c83ea640ad273c3e2:
>
> libpam: register PAM session with logind (2013-01-19 22:35:52 +0000)
>
> ----------------------------------------------------------------
> Koen Kooi (1):
> libpam: register PAM session with logind
>
> Martin Donnelly (1):
> base-files: add fstab for systemd based systems
>
> Radu Moisan (5):
> default-distrovars: Add DISTRO_FEATURES_INITMAN to DISTRO_FEATURES
> default-providers: Automatically set PREFERRED_PROVIDER_udev
> systemd: add systemd recipes
> default-providers: Add systemd option to PREFERRED_PROVIDER_udev
> update-rc.d: disable update-rc.d.bbclass when systemd enabled
>
> Ross Burton (3):
> dbus: respect systemd distro feature
> packagegroup-core-boot: install systemd-compat-units on systemd images
> packagegroup-core-boot: only install initscripts if we're using sysvinit
>
> meta/classes/update-rc.d.bbclass | 90 +-----
> meta/classes/update-rc.d_real.bbclass | 89 ++++++
> meta/conf/distro/include/default-distrovars.inc | 3 +-
> meta/conf/distro/include/default-providers.inc | 1 +
> .../base-files/base-files/fstab.systemd | 9 +
> meta/recipes-core/base-files/base-files_3.0.14.bb | 7 +-
> meta/recipes-core/dbus/dbus.inc | 27 +-
> .../packagegroups/packagegroup-core-boot.bb | 7 +-
> meta/recipes-core/systemd/systemd-compat-units.bb | 58 ++++
> .../systemd/systemd-compat-units/machineid.service | 16 +
> .../systemd-compat-units/run-postinsts.service | 17 +
> meta/recipes-core/systemd/systemd-serialgetty.bb | 45 +++
> .../systemd-serialgetty/serial-getty@.service | 32 ++
> .../systemd/systemd-systemctl-native.bb | 15 +
> .../systemd/systemd-systemctl/systemctl | 126 ++++++++
> .../systemd/systemd/00-create-volatile.conf | 10 +
> ...emd-analyze-use-argparse-instead-of-getop.patch | 103 ++++++
> ...-analyze-use-GDBus-instead-of-dbus-python.patch | 91 ++++++
> meta/recipes-core/systemd/systemd/modprobe.rules | 17 +
> .../systemd/systemd/optional_secure_getenv.patch | 19 ++
> .../systemd-pam-configure-check-uclibc.patch | 26 ++
> .../systemd/systemd/systemd-pam-fix-execvpe.patch | 26 ++
> .../systemd/systemd-pam-fix-fallocate.patch | 82 +++++
> .../systemd/systemd-pam-fix-getty-unit.patch | 35 ++
> .../systemd/systemd/systemd-pam-fix-mkostemp.patch | 166 ++++++++++
> .../systemd/systemd/systemd-pam-fix-msformat.patch | 335 ++++++++++++++++++++
> .../recipes-core/systemd/systemd/touchscreen.rules | 18 ++
> .../systemd/systemd/use-rootlibdir.patch | 107 +++++++
> meta/recipes-core/systemd/systemd/var-run.conf | 4 +
> meta/recipes-core/systemd/systemd_196.bb | 249 +++++++++++++++
> .../pam/libpam/pam.d/common-session | 4 +-
> 31 files changed, 1726 insertions(+), 108 deletions(-)
> create mode 100644 meta/classes/update-rc.d_real.bbclass
> create mode 100644 meta/recipes-core/base-files/base-files/fstab.systemd
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units.bb
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units/machineid.service
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units/run-postinsts.service
> create mode 100644 meta/recipes-core/systemd/systemd-serialgetty.bb
> create mode 100644 meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service
> create mode 100644 meta/recipes-core/systemd/systemd-systemctl-native.bb
> create mode 100755 meta/recipes-core/systemd/systemd-systemctl/systemctl
> create mode 100644 meta/recipes-core/systemd/systemd/00-create-volatile.conf
> create mode 100644 meta/recipes-core/systemd/systemd/0001-Revert-systemd-analyze-use-argparse-instead-of-getop.patch
> create mode 100644 meta/recipes-core/systemd/systemd/0002-Revert-analyze-use-GDBus-instead-of-dbus-python.patch
> create mode 100644 meta/recipes-core/systemd/systemd/modprobe.rules
> create mode 100644 meta/recipes-core/systemd/systemd/optional_secure_getenv.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-configure-check-uclibc.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-execvpe.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-fallocate.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-getty-unit.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-mkostemp.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-msformat.patch
> create mode 100644 meta/recipes-core/systemd/systemd/touchscreen.rules
> create mode 100644 meta/recipes-core/systemd/systemd/use-rootlibdir.patch
> create mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
> create mode 100644 meta/recipes-core/systemd/systemd_196.bb
>
> Koen Kooi (1):
> libpam: register PAM session with logind
>
> Martin Donnelly (1):
> base-files: add fstab for systemd based systems
>
> Radu Moisan (5):
> default-distrovars: Add DISTRO_FEATURES_INITMAN to DISTRO_FEATURES
> default-providers: Automatically set PREFERRED_PROVIDER_udev
> systemd: add systemd recipes
> default-providers: Add systemd option to PREFERRED_PROVIDER_udev
> update-rc.d: disable update-rc.d.bbclass when systemd enabled
>
> Ross Burton (3):
> dbus: respect systemd distro feature
> packagegroup-core-boot: install systemd-compat-units on systemd
> images
> packagegroup-core-boot: only install initscripts if we're using
> sysvinit
>
> meta/classes/update-rc.d.bbclass | 90 +-----
> meta/classes/update-rc.d_real.bbclass | 89 ++++++
> meta/conf/distro/include/default-distrovars.inc | 3 +-
> meta/conf/distro/include/default-providers.inc | 1 +
> .../base-files/base-files/fstab.systemd | 9 +
> meta/recipes-core/base-files/base-files_3.0.14.bb | 7 +-
> meta/recipes-core/dbus/dbus.inc | 27 +-
> .../packagegroups/packagegroup-core-boot.bb | 7 +-
> meta/recipes-core/systemd/systemd-compat-units.bb | 58 ++++
> .../systemd/systemd-compat-units/machineid.service | 16 +
> .../systemd-compat-units/run-postinsts.service | 17 +
> meta/recipes-core/systemd/systemd-serialgetty.bb | 45 +++
> .../systemd-serialgetty/serial-getty@.service | 32 ++
> .../systemd/systemd-systemctl-native.bb | 15 +
> .../systemd/systemd-systemctl/systemctl | 126 ++++++++
> .../systemd/systemd/00-create-volatile.conf | 10 +
> ...emd-analyze-use-argparse-instead-of-getop.patch | 103 ++++++
> ...-analyze-use-GDBus-instead-of-dbus-python.patch | 91 ++++++
> meta/recipes-core/systemd/systemd/modprobe.rules | 17 +
> .../systemd/systemd/optional_secure_getenv.patch | 19 ++
> .../systemd-pam-configure-check-uclibc.patch | 26 ++
> .../systemd/systemd/systemd-pam-fix-execvpe.patch | 26 ++
> .../systemd/systemd-pam-fix-fallocate.patch | 82 +++++
> .../systemd/systemd-pam-fix-getty-unit.patch | 35 ++
> .../systemd/systemd/systemd-pam-fix-mkostemp.patch | 166 ++++++++++
> .../systemd/systemd/systemd-pam-fix-msformat.patch | 335 ++++++++++++++++++++
> .../recipes-core/systemd/systemd/touchscreen.rules | 18 ++
> .../systemd/systemd/use-rootlibdir.patch | 107 +++++++
> meta/recipes-core/systemd/systemd/var-run.conf | 4 +
> meta/recipes-core/systemd/systemd_196.bb | 249 +++++++++++++++
> .../pam/libpam/pam.d/common-session | 4 +-
> 31 files changed, 1726 insertions(+), 108 deletions(-)
> create mode 100644 meta/classes/update-rc.d_real.bbclass
> create mode 100644 meta/recipes-core/base-files/base-files/fstab.systemd
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units.bb
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units/machineid.service
> create mode 100644 meta/recipes-core/systemd/systemd-compat-units/run-postinsts.service
> create mode 100644 meta/recipes-core/systemd/systemd-serialgetty.bb
> create mode 100644 meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service
> create mode 100644 meta/recipes-core/systemd/systemd-systemctl-native.bb
> create mode 100755 meta/recipes-core/systemd/systemd-systemctl/systemctl
> create mode 100644 meta/recipes-core/systemd/systemd/00-create-volatile.conf
> create mode 100644 meta/recipes-core/systemd/systemd/0001-Revert-systemd-analyze-use-argparse-instead-of-getop.patch
> create mode 100644 meta/recipes-core/systemd/systemd/0002-Revert-analyze-use-GDBus-instead-of-dbus-python.patch
> create mode 100644 meta/recipes-core/systemd/systemd/modprobe.rules
> create mode 100644 meta/recipes-core/systemd/systemd/optional_secure_getenv.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-configure-check-uclibc.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-execvpe.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-fallocate.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-getty-unit.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-mkostemp.patch
> create mode 100644 meta/recipes-core/systemd/systemd/systemd-pam-fix-msformat.patch
> create mode 100644 meta/recipes-core/systemd/systemd/touchscreen.rules
> create mode 100644 meta/recipes-core/systemd/systemd/use-rootlibdir.patch
> create mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
> create mode 100644 meta/recipes-core/systemd/systemd_196.bb
>
[-- Attachment #2: Type: text/html, Size: 12727 bytes --]
next prev parent reply other threads:[~2013-01-21 3:43 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 [this message]
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
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=1358739008.2945.29.camel@jethro.cheepee.homedns.org \
--to=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox