| Submitter | Ross Burton |
|---|---|
| Date | Jan. 19, 2013, 10:47 p.m. |
| Message ID | <cover.1358635193.git.ross.burton@intel.com> |
| Download | mbox |
| Permalink | /patch/42935/ |
| State | New |
| Headers | show |
Pull-request
git://git.yoctoproject.org/poky-contrib ross/systemd-minimalComments
On 01/19/2013 02:47 PM, 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. > This change introduced a new warning about un-packaged files: > WARNING: QA Issue: connman: Files/directories were installed but not shipped > /lib > /lib/systemd > /lib/systemd/system > /lib/systemd/system/connman.service Sau! > 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 >
On 20 January 2013 18:34, Saul Wold <sgw@linux.intel.com> wrote: > This change introduced a new warning about un-packaged files: > >> WARNING: QA Issue: connman: Files/directories were installed but not >> shipped >> /lib >> /lib/systemd >> /lib/systemd/system >> /lib/systemd/system/connman.service That will be connman detecting the systemd pkgconfig files in the sysroot and enabling the integration automatically. To avoid either a) enabling systemd integration in every service that does this or b) disabling systemd in every service that does this, can we ignore these warnings knowing that they'll be resolved when the rest of the integration lands? Ross
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 >
On 01/20/2013 10:34 AM, Saul Wold wrote: > On 01/19/2013 02:47 PM, 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. >> > > This change introduced a new warning about un-packaged files: > >> WARNING: QA Issue: connman: Files/directories were installed but not >> shipped >> /lib >> /lib/systemd >> /lib/systemd/system >> /lib/systemd/system/connman.service > > > Sau! > > >> 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 >> Another thing I just noticed is that many of these patches are missing either Signed-off-by or Upstream-Status Tags! Thanks Sau! >> 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 >> > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
The autobuilder hit this last night: ERROR: Multiple .bb files are due to be built which each provide udev (/srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/udev/udev_182.bb /srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/systemd/systemd_196.bb). Which I realised would happen not long after hitting send. I'll sort this too. Ross
Hi Ross, Le Mon, 21 Jan 2013 08:08:32 +0000, "Burton, Ross" <ross.burton@intel.com> a écrit : > The autobuilder hit this last night: > > ERROR: Multiple .bb files are due to be built which each provide udev > (/srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/udev/udev_182.bb > /srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/systemd/systemd_196.bb). > > Which I realised would happen not long after hitting send. I'll sort this too. > we had the same problem in meta-oe's denzil branch. The workaround is here : http://patches.openembedded.org/patch/41669/ http://patches.openembedded.org/patch/41667/ http://patches.openembedded.org/patch/41671/ What should work in OE-Core when using systemd is : PREFERRED_PROVIDER_udev = "systemd" Eric
On 01/21/2013 10:19 AM, Eric Bénard wrote: > Hi Ross, > > Le Mon, 21 Jan 2013 08:08:32 +0000, > "Burton, Ross" <ross.burton@intel.com> a écrit : > >> The autobuilder hit this last night: >> >> ERROR: Multiple .bb files are due to be built which each provide udev >> (/srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/udev/udev_182.bb >> /srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/systemd/systemd_196.bb). >> >> Which I realised would happen not long after hitting send. I'll sort this too. >> > we had the same problem in meta-oe's denzil branch. > > The workaround is here : > http://patches.openembedded.org/patch/41669/ > http://patches.openembedded.org/patch/41667/ > http://patches.openembedded.org/patch/41671/ > > What should work in OE-Core when using systemd is : > PREFERRED_PROVIDER_udev = "systemd" This is addressed in [PATCH 2/10] & [PATCH 5/10], it should not require specifically setting the PREFERRED_PROVIDER_udev Radu
On Sun, Jan 20, 2013 at 08:21:18PM +0000, Burton, Ross wrote: > On 20 January 2013 18:34, Saul Wold <sgw@linux.intel.com> wrote: > > This change introduced a new warning about un-packaged files: > > > >> WARNING: QA Issue: connman: Files/directories were installed but not > >> shipped > >> /lib > >> /lib/systemd > >> /lib/systemd/system > >> /lib/systemd/system/connman.service > > That will be connman detecting the systemd pkgconfig files in the > sysroot and enabling the integration automatically. To avoid either > a) enabling systemd integration in every service that does this or b) > disabling systemd in every service that does this, can we ignore these > warnings knowing that they'll be resolved when the rest of the > integration lands? Why weren't changes from meta-systemd/oe-core/recipes-connectivity/connman/connman_1.10.bbappend merged to oe-core? Cheers,
On 21 January 2013 08:59, Martin Jansa <martin.jansa@gmail.com> wrote: > Why weren't changes from > meta-systemd/oe-core/recipes-connectivity/connman/connman_1.10.bbappend > merged to oe-core? Because I'm 90% sure connman isn't the only package that does runtime detection of systemd. As I said in the original mail, the goal of this series was to merge the core support, not all of the services. Mainly, I want to understand systemd.bbclass before submitting it. Ross
On Mon, 2013-01-21 at 09:19 +0100, Eric Bénard wrote: > Hi Ross, > > Le Mon, 21 Jan 2013 08:08:32 +0000, > "Burton, Ross" <ross.burton@intel.com> a écrit : > > > The autobuilder hit this last night: > > > > ERROR: Multiple .bb files are due to be built which each provide udev > > (/srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/udev/udev_182.bb > > /srv/home/pokybuild/yocto-autobuilder/yocto-slave/nightly-world/build/meta/recipes-core/systemd/systemd_196.bb). > > > > Which I realised would happen not long after hitting send. I'll sort this too. > > > we had the same problem in meta-oe's denzil branch. > > The workaround is here : > http://patches.openembedded.org/patch/41669/ > http://patches.openembedded.org/patch/41667/ > http://patches.openembedded.org/patch/41671/ > > What should work in OE-Core when using systemd is : > PREFERRED_PROVIDER_udev = "systemd" This isn't enough to prevent warnings for "bitbake world". We need to detect which is enabled and then SkipPackage in the one that isn't needed, or at least exclude it from the world build. Cheers, Richard
On 21 January 2013 10:14, Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > This isn't enough to prevent warnings for "bitbake world". We need to > detect which is enabled and then SkipPackage in the one that isn't > needed, or at least exclude it from the world build. Patches sent so udev and systemd skip if the other is going to be built. Ross
On 21 January 2013 03:57, Saul Wold <sgw@linux.intel.com> wrote: > Another thing I just noticed is that many of these patches are missing > either Signed-off-by or Upstream-Status Tags! Yes. I meant to wipe out the uclibc patches for now to reduce the attribution-chasing, but forgot. I'll work on that next. Ross
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). 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. Ross
On 01/21/2013 04:12 AM, 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). > > 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. > Ouch, I thought that this was one of the goals of the select your init system, ie don't use either sysvinit or systemd. Poky-Tiny has a single file rc.local script to do the initialization of the system. Sau! > Ross > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core > >
On 21 January 2013 16:57, Saul Wold <sgw@linux.intel.com> wrote: > Ouch, I thought that this was one of the goals of the select your init > system, ie don't use either sysvinit or systemd. Poky-Tiny has a single > file rc.local script to do the initialization of the system. And presumably a poky-tiny image doesn't pull in any actual services, as they'd have been installing /etc/init.d/* files and invoking update-rc.d. Well, as of yesterday, that is. Now that update-rc.d is a no-op without sysvinit, at least the postinsts don't happen. Ross
On 01/20/2013 06:47 AM, 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 > > Maybe we need to add something like below to meta-yocto/conf/local.conf.sample ? # set default init manager: sysvinit or systemd DISTRO_FEATURES_INITMAN ?= "sysvinit" This may help make it clearer how to choose an init manager in conf/local.conf file. Cheers, Chen Qi
On 01/22/2013 11:30 AM, ChenQi wrote: > On 01/20/2013 06:47 AM, 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 >> >> > > Maybe we need to add something like below to > meta-yocto/conf/local.conf.sample ? > > # set default init manager: sysvinit or systemd > DISTRO_FEATURES_INITMAN ?= "sysvinit" > > This may help make it clearer how to choose an init manager in > conf/local.conf file. Sounds good to me. Radu
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? > 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? I guess the most important aspect I am trying to communicate is: please do not provide any by default. Cipi > Ross
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
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