udev is now provided by systemd or eudev

eudev is a userspace device management daemon. It is a standalone version, independent from systemd. It is a fork maintained by Gentoo.

Features:

- No extra configuration options are available: Gudev is build if libglib2 is selected.
- No dependency on hwdata as the package uses its own hardware database (as does systemd).

eudev 1.3 is in sync with systemd v207.

Signed-off-by Eric Le Bihan <[email protected]>

package/eudev/Config.in new file +27 -0

new file mode 100644
index 0000000..94d94e0
@@ -0,0 +1,37 @@
+config BR2_PACKAGE_EUDEV
+	bool "eudev"
+	depends on !BR2_avr32 # no epoll_create1
+	depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV
+	depends on BR2_LARGEFILE # util-linux
+	depends on BR2_USE_WCHAR # util-linux
+	depends on !BR2_PREFER_STATIC_LIB # kmod
+	select BR2_PACKAGE_UTIL_LINUX
+	select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
+	select BR2_PACKAGE_KMOD
+	help
+	  Userspace device daemon. This is a standalone version,
+	  independent of systemd. It is a fork maintained by Gentoo.
+
+	  eudev requires a Linux kernel >= 2.6.34: it relies on devtmpfs
+	  and inotify.
+
+	  http://github.com/gentoo/eudev/
+
+if BR2_PACKAGE_EUDEV
+
+config BR2_PACKAGE_EUDEV_RULES_GEN
+	bool "enable rules generator"
+	help
+	  Enable persistent rules generator
+
+endif

Quoted 19 lines

package/eudev/Config.in new file +13 -0

new file mode 100644
index 0000000..94d94e0
@@ -0,0 +1,37 @@
+config BR2_PACKAGE_EUDEV
+	bool "eudev"
+	depends on !BR2_avr32 # no epoll_create1
+	depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV
+	depends on BR2_LARGEFILE # util-linux
+	depends on BR2_USE_WCHAR # util-linux
+	depends on !BR2_PREFER_STATIC_LIB # kmod
+	select BR2_PACKAGE_UTIL_LINUX
+	select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
+	select BR2_PACKAGE_KMOD
+	help
+	  Userspace device daemon. This is a standalone version,
+	  independent of systemd. It is a fork maintained by Gentoo.

package/eudev/Config.in line 18

Quoted 5 lines

package/eudev/Config.in new file +5 -0

+
+	  eudev requires a Linux kernel >= 2.6.34: it relies on devtmpfs
+	  and inotify.
+
+	  http://github.com/gentoo/eudev/

I don't see the mmu option on the main symbol, so either that or the comment dependencies are not correct..

This patch converts udev to a virtual package. For the moment, there is only one provider for the udev features: eudev.

Packages meant to provide udev-like features must select the symbol BR2_PACKAGE_HAS_UDEV.

Packages depending on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV or BR2_PACKAGE_UDEV have been converted to use the new symbol.

Signed-off-by Eric Le Bihan <[email protected]>

---
 Config.in.legacy                                   |   55 ++++++++++++++++++++
 docs/manual/adding-packages-directory.txt          |    3 +-
 linux/linux.mk                                     |    2 -
 package/ccid/ccid.mk                               |    2 +-
 package/eudev/Config.in                            |    1 +
 package/gpsd/gpsd.mk                               |    2 +-
 package/libatasmart/Config.in                      |    4 +-
 package/libcec/libcec.mk                           |    2 +-
 package/libdrm/libdrm.mk                           |    2 +-
 package/libmbim/Config.in                          |    5 +-
 package/libusb/libusb.mk                           |    2 +-
 package/modem-manager/Config.in                    |    6 +--
 package/network-manager/Config.in                  |    8 ++-
 package/ofono/ofono.mk                             |    2 +-
 package/pcsc-lite/Config.in                        |    2 +-
 package/pcsc-lite/pcsc-lite.mk                     |    2 +-
 package/pulseaudio/pulseaudio.mk                   |    2 +-
 package/systemd/Config.in                          |    6 +--
 package/udev/Config.in                             |   51 ++----------------
 package/udev/udev.mk                               |   49 ++++-------------
 package/udisks/Config.in                           |   14 ++---
 package/usbmount/Config.in                         |    4 +-
 package/vlc/vlc.mk                                 |    2 +-
 package/weston/Config.in                           |    4 +-
 package/x11r7/xdriver_xf86-input-evdev/Config.in   |    4 +-
 .../xserver_xorg-server/xserver_xorg-server.mk     |    2 +-
 package/xenomai/xenomai.mk                         |    2 +-
 system/Config.in                                   |   22 ++------
 28 files changed, 112 insertions(+), 150 deletions(-)

Config.in.legacy +3 -0

index 2fe9cf9..6d594c6 100644
@@ -101,6 +101,61 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2014.02"
 
+config BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+       bool "/dev management by udev removed"
+       select BR2_LEGACY

Config.in.legacy +42 -0

+
+config BR2_PACKAGE_UDEV
+       bool "udev is now a virtual package"
+       select BR2_LEGACY
+       select BR2_PACKAGE_HAS_UDEV
+       help
+         The 'udev' package has been converted to a virtual package.
+         There is currently one provider for this feature: 'eudev'.
+
+         Your old configuration refers to packages depending on 'udev',
+         either for build or at runtime.
+
+         Check that a 'udev' provider is selected, i.e. that 'eudev' is
+         selected, which is the case if '/dev management' is set to
+         'Dynamic using eudev'.
+
+config BR2_PACKAGE_UDEV_RULES_GEN
+       bool "udev rules generation handled by eudev"
+       select BR2_LEGACY
+       select BR2_PACKAGE_EUDEV
+       select BR2_PACKAGE_EUDEV_RULES_GEN
+       help
+         The 'udev' package has been converted to a virtual package.
+         There is currently one provider for this feature: 'eudev'.
+
+         udev rules generation will now be handled by 'eudev'.
+
+         Check that '/dev management' is set to 'Dynamic using eudev'
+         to get the same behaviour as in your old configuration.
+
+config BR2_PACKAGE_UDEV_ALL_EXTRAS
+       bool "udev extras removed"
+       select BR2_LEGACY
+       help
+         The 'udev' package has been converted to a virtual package.
+         There is currently one provider for this feature: 'eudev'.
+
+         The option to enable the extra features of 'udev' (gudev, ...)
+         has been removed. These features are automatically enabled in
+         the 'udev' providers if the dependencies are selected. For
+         example, selecting 'libglib2' will trigger the build of gudev.
+
 config BR2_sh2
        bool "sh2 support removed"
        help

package/libatasmart/Config.in +1 -1

index a3e44f8..534e7c1 100644
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_LIBATASMART
        bool "libatasmart"
-       depends on BR2_PACKAGE_UDEV # libudev is configure dependency
+       depends on BR2_PACKAGE_HAS_UDEV # libudev is configure dependency
        help
          The libatasmart package is a disk reporting library.
          It only supports a subset of the ATA S.M.A.R.T. functionality.

package/udev/Config.in +2 -39

index 7aa79c4..5cd4051 100644
@@ -1,47 +1,6 @@
-config BR2_PACKAGE_UDEV
-       bool "udev"
-       depends on !BR2_avr32 # no epoll_create1
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
-       depends on BR2_LARGEFILE # util-linux
-       depends on BR2_USE_WCHAR # util-linux
-       depends on !BR2_PREFER_STATIC_LIB # kmod
-       select BR2_PACKAGE_UTIL_LINUX
-       select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
-       select BR2_PACKAGE_KMOD
+config BR2_PACKAGE_HAS_UDEV
+       bool
        help
-         Userspace device daemon.
-
-         udev requires a Linux kernel >= 2.6.34: it relies on devtmpfs
-         and inotify.
-
-         ftp://ftp.kernel.org/pub/linux/utils/kernel/hotplug/
-
-if BR2_PACKAGE_UDEV
-
-config BR2_PACKAGE_UDEV_RULES_GEN
-       bool "enable rules generator"
-       help
-         Enable persistant rules generator
-
-config BR2_PACKAGE_UDEV_ALL_EXTRAS
-       bool "enable all extras"
-       select BR2_PACKAGE_ACL
-       select BR2_PACKAGE_HWDATA
-       select BR2_PACKAGE_LIBGLIB2
-       depends on BR2_USE_WCHAR # libglib2
-       depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
-       depends on BR2_USE_MMU # libglib2
-       help
-         Enable all extras with external dependencies like
-         libacl, hwdata and libglib2
-
-comment "enabling all extras needs a toolchain w/ wchar, threads"
-       depends on BR2_USE_MMU
-       depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS

Quoted 17 lines

Config.in.legacy +9 -0

index 2fe9cf9..6d594c6 100644
@@ -101,6 +101,61 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2014.02"
 
+config BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+       bool "/dev management by udev removed"
+       select BR2_LEGACY
+       help
+         The 'udev' package has been converted to a virtual package.
+         There is currently one provider for this feature: 'eudev'.
+
+         Therefore, you must choose 'Dynamic using eudev' in the
+         '/dev management' menu to get the same behaviour as in your

Config.in.legacy endif · line 116

Quoted 4 lines

Config.in.legacy +4 -0

+          old configuration.
+
+         You must also check the packages depending on 'udev' are still
+         selected.

Is there a particular reason why you don't automatically select the suggested alternative?
This will make the transition easier for users...

Quoted 15 lines

package/libatasmart/Config.in +1 -1

index a3e44f8..534e7c1 100644
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_LIBATASMART
        bool "libatasmart"
-       depends on BR2_PACKAGE_UDEV # libudev is configure dependency
+       depends on BR2_PACKAGE_HAS_UDEV # libudev is configure dependency
        help
          The libatasmart package is a disk reporting library.
          It only supports a subset of the ATA S.M.A.R.T. functionality.

@@ -8,4 +8,4 @@ config BR2_PACKAGE_LIBATASMART
          http://www.linuxfromscratch.org/blfs/view/svn/general/libatasmart.html
 

package/libatasmart/Config.in config BR2_PACKAGE_LIBATASMART · line 11

Quoted 3 lines

package/libatasmart/Config.in +1 -1

 comment "libatasmart requires udev to be enabled"
-       depends on !BR2_PACKAGE_UDEV
+       depends on !BR2_PACKAGE_HAS_UDEV

Finally we should also line up all the udev comments, but this can be done in a separate patch.

package/libatasmart/Config.in

Quoted 3 lines

package/libatasmart/Config.in +3 -0

+         udev is now a virtual package.
+         Packages requiring udevd at run time or libudev/gudev at build time
+         should select it.

Does it make sense to adding a help option on a hidden option? Can the user ever make it visible?

Quoted 18 lines

Quoted 18 lines

Config.in.legacy +9 -0

index 2fe9cf9..6d594c6 100644
@@ -101,6 +101,61 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2014.02"
 
+config BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+       bool "/dev management by udev removed"
+       select BR2_LEGACY
+       help
+         The 'udev' package has been converted to a virtual package.
+         There is currently one provider for this feature: 'eudev'.
+
+         Therefore, you must choose 'Dynamic using eudev' in the
+         '/dev management' menu to get the same behaviour as in your

Quoted 8 lines

Config.in.legacy endif · line 116

Quoted 4 lines

Config.in.legacy +4 -0

+          old configuration.
+
+         You must also check the packages depending on 'udev' are still
+         selected.

Is there a particular reason why you don't automatically select the suggested alternative?
This will make the transition easier for users...

That's what I did on first try. BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV was selected. But I ended up facing a limit of kconfig: this boolean symbol is used in a choice/endchoice ("/dev management"), which prevents its value from being recalculated when it is selected from Config.in.legacy.

For example, if BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV=y in test_udev_legacy_defconfig and we perform:

  $ make test_udev_legacy_defconfig

output/.config will contain:

# BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV is not set BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_DEVTMPFS=y

The default choice is selected. I discussed this issue on IRC with Yann E. Morin last evening, but there is no workaround... So I chose to remove the automatic selection of the new symbol and add detailed instructions.

package/modem-manager/Config.in +1 -1

index e157ca8..fe60850 100644
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_MODEM_MANAGER
        bool "modemmanager"
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+       depends on BR2_PACKAGE_HAS_UDEV
        select BR2_PACKAGE_DBUS
        depends on BR2_INET_IPV6
        depends on BR2_LARGEFILE # acl

package/modem-manager/Config.in +1 -1

        help
          ModemManager is a DBus-activated daemon which controls mobile
          broadband (2G/3G/4G) devices and connections.

@@ -33,4 +31,4 @@ endif
 
 comment "modemmanager needs udev /dev management and a toolchain w/ largefile, wchar, threads, IPv6"
        depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
-               !BR2_TOOLCHAIN_HAS_THREADS || !BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+               !BR2_TOOLCHAIN_HAS_THREADS || !BR2_PACKAGE_HAS_UDEV

package/network-manager/Config.in +0 -2

index a194433..a4eeeaf 100644
@@ -1,16 +1,14 @@
 config BR2_PACKAGE_NETWORK_MANAGER
        bool "NetworkManager"
        depends on !BR2_avr32 # udev
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
-       select BR2_PACKAGE_DBUS

package/network-manager/Config.in +1 -1

        select BR2_PACKAGE_GNUTLS
        select BR2_PACKAGE_LIBGCRYPT
        select BR2_PACKAGE_LIBNL

@@ -33,4 +31,4 @@ comment "NetworkManager needs udev /dev management and a toolchain w/ IPv6, larg
        depends on BR2_USE_MMU
        depends on !BR2_INET_IPV6 || !BR2_LARGEFILE || !BR2_USE_WCHAR || \
                !BR2_TOOLCHAIN_HAS_THREADS || \
-               !BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+               !BR2_PACKAGE_HAS_UDEV

package/udev/Config.in +2 -39

index 7aa79c4..5cd4051 100644
@@ -1,47 +1,6 @@
-config BR2_PACKAGE_UDEV
-       bool "udev"
-       depends on !BR2_avr32 # no epoll_create1
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
-       depends on BR2_LARGEFILE # util-linux
-       depends on BR2_USE_WCHAR # util-linux
-       depends on !BR2_PREFER_STATIC_LIB # kmod
-       select BR2_PACKAGE_UTIL_LINUX
-       select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
-       select BR2_PACKAGE_KMOD
+config BR2_PACKAGE_HAS_UDEV
+       bool
        help
-         Userspace device daemon.
-
-         udev requires a Linux kernel >= 2.6.34: it relies on devtmpfs
-         and inotify.
-
-         ftp://ftp.kernel.org/pub/linux/utils/kernel/hotplug/
-
-if BR2_PACKAGE_UDEV
-
-config BR2_PACKAGE_UDEV_RULES_GEN
-       bool "enable rules generator"
-       help
-         Enable persistant rules generator
-
-config BR2_PACKAGE_UDEV_ALL_EXTRAS
-       bool "enable all extras"
-       select BR2_PACKAGE_ACL
-       select BR2_PACKAGE_HWDATA
-       select BR2_PACKAGE_LIBGLIB2
-       depends on BR2_USE_WCHAR # libglib2
-       depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
-       depends on BR2_USE_MMU # libglib2
-       help
-         Enable all extras with external dependencies like
-         libacl, hwdata and libglib2
-
-comment "enabling all extras needs a toolchain w/ wchar, threads"
-       depends on BR2_USE_MMU
-       depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS

package/udev/udev.mk +3 -32

index db86850..d274246 100644
@@ -4,48 +4,17 @@
 #
 ################################################################################
 
-UDEV_VERSION = 182
-UDEV_SOURCE = udev-$(UDEV_VERSION).tar.xz
-UDEV_SITE = $(BR2_KERNEL_MIRROR)/linux/utils/kernel/hotplug/
-UDEV_LICENSE = GPLv2+
-UDEV_LICENSE_FILES = COPYING
-UDEV_INSTALL_STAGING = YES
+UDEV_SOURCE =
 
-# mq_getattr is in librt
-UDEV_CONF_ENV += LIBS=-lrt
-
-UDEV_CONF_OPT =                        \
-       --sbindir=/sbin         \
-       --with-rootlibdir=/lib  \
-       --libexecdir=/lib       \
-       --with-usb-ids-path=/usr/share/hwdata/usb.ids   \
-       --with-pci-ids-path=/usr/share/hwdata/pci.ids   \
-       --with-firmware-path=/lib/firmware              \
-       --disable-introspection
-
-UDEV_DEPENDENCIES = host-gperf host-pkgconf util-linux kmod
-
-ifeq ($(BR2_PACKAGE_UDEV_RULES_GEN),y)
-UDEV_CONF_OPT += --enable-rule_generator
-endif
-
-ifeq ($(BR2_PACKAGE_UDEV_ALL_EXTRAS),y)
-UDEV_DEPENDENCIES += acl hwdata libglib2
-UDEV_CONF_OPT +=               \
-       --enable-udev_acl
-else
-UDEV_CONF_OPT +=               \
-       --disable-gudev
+ifeq ($(BR2_PACKAGE_EUDEV),y)
+UDEV_DEPENDENCIES += eudev
 endif
 
-ifeq ($(BR2_PACKAGE_SYSTEMD),y)

package/udev/udev.mk +2 -3

+endif
 
-UDEV_POST_INSTALL_TARGET_HOOKS += UDEV_INSTALL_INITSCRIPT
-
-$(eval $(autotools-package))
+$(eval $(generic-package))

package/udisks/Config.in +0 -0

index e9539a3..d560b6c 100644

package/udisks/Config.in +1 -1

        select BR2_PACKAGE_DBUS
        select BR2_PACKAGE_DBUS_GLIB
        depends on BR2_USE_WCHAR # dbus-glib -> glib2

@@ -36,8 +34,12 @@ config BR2_PACKAGE_UDISKS_LVM2
 
 endif
 
-comment "udisks needs udev /dev management and a toolchain w/ wchar, threads"
+comment "udisks needs udev /dev management"

Quoted 16 lines

package/modem-manager/Config.in +1 -1

index e157ca8..fe60850 100644
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_MODEM_MANAGER
        bool "modemmanager"
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
+       depends on BR2_PACKAGE_HAS_UDEV
        select BR2_PACKAGE_DBUS
        depends on BR2_INET_IPV6
        depends on BR2_LARGEFILE # acl

@@ -8,8 +8,6 @@ config BR2_PACKAGE_MODEM_MANAGER
        depends on BR2_TOOLCHAIN_HAS_THREADS # dbus, libglib2
        depends on BR2_USE_MMU # dbus
        select BR2_PACKAGE_DBUS_GLIB

package/modem-manager/Config.in config BR2_PACKAGE_MODEM_MANAGER · line 12

Quoted 2 lines

package/modem-manager/Config.in +0 -2

-       select BR2_PACKAGE_UDEV
-       select BR2_PACKAGE_UDEV_ALL_EXTRAS

When removing BR2_PACKAGE_UDEV_ALL_EXTRAS, you should certainly replace it by the
"select ..." statements done in the BR2_PACKAGE_UDEV_ALL_EXTRAS symbol, ie: acl and libglib2 (if I understood correctly hwdata is no longer needed).

This patch bumps systemd to v207 but also declares it as a provider for the udev virtual package.

Starting with systemd 183, udev has been merged into systemd. The udev daemon is now installed as /lib/systemd/systemd-udevd. This means that /dev management using udev is only available if systemd is chosen as init system.

When configuring systemd, the following options are available:

- activation of systemd-journal-gatewayd, to access the journal via HTTP.
- activation of extra features like journal compression and sealing.

Support for uClibc has also been removed because:

- upstream has no interest in supporting uClibc.
- using a shrinked libc brings no advantage, given the size of all the programs included in Systemd. So using glibc does not matter.

Signed-off-by Eric Le Bihan <[email protected]>

package/systemd/Config.in +11 -1

index 500e623..e3ea433 100644
@@ -1,11 +1,21 @@
 config BR2_PACKAGE_SYSTEMD
        bool "systemd"
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV
+       depends on BR2_INIT_SYSTEMD
+       depends on !BR2_avr32 # no epoll_create1
+       depends on BR2_LARGEFILE # util-linux
+       depends on BR2_USE_WCHAR # util-linux
        depends on BR2_INET_IPV6
+       depends on !BR2_PREFER_STATIC_LIB # kmod
        depends on BR2_TOOLCHAIN_HAS_THREADS # dbus
        depends on BR2_USE_MMU # dbus
+       select BR2_PACKAGE_HAS_UDEV
        select BR2_PACKAGE_DBUS
        select BR2_PACKAGE_LIBCAP
+       select BR2_PACKAGE_UTIL_LINUX
+       select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
+       select BR2_PACKAGE_KMOD
+       select BR2_PACKAGE_BUSYBOX_SHOW_OTHERS # kmod-tools
+       select BR2_PACKAGE_KMOD_TOOLS
        help
          systemd is a system and service manager for Linux, compatible with
          SysV and LSB init scripts. systemd provides aggressive parallelization

package/systemd/Config.in +29 -4

+
+         Systemd also provides udev, the userspace device daemon.
+
+         The selection of other packages will enable some features:
+
+         - libglib2 package will add support for gudev.
+         - acl package will add support for multi-seat.
+
          http://freedesktop.org/wiki/Software/systemd
 
-comment "systemd needs eudev /dev management and a toolchain w/ IPv6, threads"
-       depends on BR2_USE_MMU
-       depends on !BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV || !BR2_INET_IPV6 || \\
-               !BR2_TOOLCHAIN_HAS_THREADS
+if BR2_PACKAGE_SYSTEMD
+
+config BR2_PACKAGE_SYSTEMD_ALL_EXTRAS
+       bool "enable all extras"
+       select BR2_PACKAGE_XZ
+       select BR2_PACKAGE_LIBGCRYPT
+       help
+         Enable extra features for Systemd: journal compression and
+         signing.
+
+config BR2_PACKAGE_SYSTEMD_JOURNAL_GATEWAY
+       bool "HTTP server for journal events"
+       select BR2_PACKAGE_LIBMICROHTTPD
+       help
+         systemd-journal-gatewayd serves journal events over the
+         network. Clients must connect using HTTP. The server
+         listens on port 19531 by default.
+
+         http://www.freedesktop.org/software/systemd/man/systemd-journal-gatewayd.service.html
+
+endif

system/Config.in +2 -0

index 34726b0..30d8efe 100644
@@ -87,20 +87,19 @@ config BR2_INIT_SYSV
 
 config BR2_INIT_SYSTEMD
        bool "systemd"
+       depends on !BR2_avr32 # no epoll_create1
+       depends on BR2_TOOLCHAIN_USES_GLIBC

system/Config.in +6 -7

-comment 'systemd needs eudev /dev management and a toolchain w/ largefile, wchar, IPv6, threads'
-       depends on BR2_USE_MMU
-       depends on !(BR2_LARGEFILE && BR2_USE_WCHAR && \
-                    BR2_INET_IPV6 && BR2_TOOLCHAIN_HAS_THREADS && \
-                    BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV)
+comment 'systemd needs an (e)glibc toolchain'
+       depends on !BR2_TOOLCHAIN_USES_GLIBC
 
 config BR2_INIT_NONE
        bool "None"

@@ -108,7 +107,7 @@ config BR2_INIT_NONE
 endchoice
 
 choice
-       prompt "/dev management"
+       prompt "/dev management" if !BR2_INIT_SYSTEMD
        default BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_DEVTMPFS
 
 config BR2_ROOTFS_DEVICE_CREATION_STATIC

@@ -135,6 +134,9 @@ comment "eudev needs a toolchain w/ largefile, wchar, dynamic library"
 
 endchoice
 
+comment "/dev management using udev (from systemd)"
+       depends on BR2_INIT_SYSTEMD
+
 config BR2_ROOTFS_DEVICE_TABLE
        string "Path to the permission tables"
        default "system/device_table.txt"
--

1.7.9.5

package/systemd/Config.in config BR2_PACKAGE_SYSTEMD · line 30

Quoted 2 lines

package/systemd/Config.in +2 -0

+         Systemd requires a Linux kernel >= 3.0, with inotify, devtmpfs,
+         tmpfs vfs and tmpfs POSIX ACL enabled.

Linux >= 3.0, really? IIRC systemd requires devtmpfs (available for a long time), and cgroup which has been available since linux-2.6.36 (though the patch to make them available on older kernel is easy to backport on older kernel).

Quoted 17 lines

system/Config.in +3 -1

index 34726b0..30d8efe 100644
@@ -87,20 +87,19 @@ config BR2_INIT_SYSV
 
 config BR2_INIT_SYSTEMD
        bool "systemd"
+       depends on !BR2_avr32 # no epoll_create1
+       depends on BR2_TOOLCHAIN_USES_GLIBC
        depends on BR2_LARGEFILE
        depends on BR2_USE_WCHAR
        depends on BR2_INET_IPV6
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV
        depends on BR2_TOOLCHAIN_HAS_THREADS
+       depends on BR2_TOOLCHAIN_HAS_SSP
        depends on BR2_USE_MMU

system/Config.in config BR2_INIT_SYSV · line 99

Quoted 3 lines

system/Config.in +1 -1

-       select BR2_PACKAGE_DBUS
+       depends on !BR2_PREFER_STATIC_LIB
        select BR2_PACKAGE_SYSTEMD

How does this play with kconfig?
Does not kconfig complain about some circular dependencies here, because you have BR2_INIT_SYSTEMD selecting BR2_PACAKGE_SYSTEMD, while BR2_PACKAGE_SYSTEMD depends on BR2_INIT_SYSTEMD?

Samuel, all,

Quoted 7 lines · Samuel Martin

package/systemd/Config.in config BR2_PACKAGE_SYSTEMD · line 30

Quoted 2 lines

package/systemd/Config.in +2 -0

+         Systemd requires a Linux kernel >= 3.0, with inotify, devtmpfs,
+         tmpfs vfs and tmpfs POSIX ACL enabled.

Linux >= 3.0, really? IIRC systemd requires devtmpfs (available for a long time), and cgroup which has been available since linux-2.6.36 (though the patch to make them available on older kernel is easy to backport on older kernel).

I set the restriction to "Linux >= 3.0" as it is the one mentioned in the README file of systemd. Linux 3.0 introduced the cgroup.procs file, needed by systemd v207.

What should the help menu contain? Should it be only the minimum version required with its reason, or the full list of features needed?

The README file contains a list of the kernel configuration options that are needed to run systemd:

- CONFIG_DEVTMPFS
- CONFIG_CGROUPS
- CONFIG_INOTIFY_USER
- CONFIG_SIGNALFD
- CONFIG_TIMERFD
- CONFIG_EPOLL
- CONFIG_NET
- CONFIG_SYSFS
- CONFIG_PROC_FS
- CONFIG_FHANDLE

Others are strongly recommended:

- CONFIG_IPV6
- CONFIG_AUTOFS4_FS
- CONFIG_TMPFS_POSIX_ACL
- CONFIG_TMPFS_XATTR
- CONFIG_SECCOMP

The LINUX_CONFIGURE_CMDS in linux/linux.mk forces some kernel options for some packages. Should I add all the ones previously listed for systemd?

I'm in favor of setting the help text to:

Systemd requires a Linux kernel >= 3.0.

The following features will be enabled in the Linux kernel:
devtmpfs, cgroups, [...]

So the end user is informed and will have a working configuration out of the box.

Quoted 18 lines

Quoted 18 lines

system/Config.in +3 -1

index 34726b0..30d8efe 100644
@@ -87,20 +87,19 @@ config BR2_INIT_SYSV
 
 config BR2_INIT_SYSTEMD
        bool "systemd"
+       depends on !BR2_avr32 # no epoll_create1
+       depends on BR2_TOOLCHAIN_USES_GLIBC
        depends on BR2_LARGEFILE
        depends on BR2_USE_WCHAR
        depends on BR2_INET_IPV6
-       depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_EUDEV
        depends on BR2_TOOLCHAIN_HAS_THREADS
+       depends on BR2_TOOLCHAIN_HAS_SSP
        depends on BR2_USE_MMU

Quoted 8 lines

system/Config.in config BR2_INIT_SYSV · line 99

Quoted 3 lines

system/Config.in +1 -1

-       select BR2_PACKAGE_DBUS
+       depends on !BR2_PREFER_STATIC_LIB
        select BR2_PACKAGE_SYSTEMD

How does this play with kconfig?
Does not kconfig complain about some circular dependencies here, because you have BR2_INIT_SYSTEMD selecting BR2_PACAKGE_SYSTEMD, while BR2_PACKAGE_SYSTEMD depends on BR2_INIT_SYSTEMD?

I do not get any error from kconfig. I used the same trick to hide the systemv package when 'SysV' is not chosen as init system. I do not know if it is a bug or a feature of kconfig :)

This patch updates the /dev management section in the manual with information about eudev, which replaces udev.

Signed-off-by Eric Le Bihan <[email protected]>

---
 docs/manual/configure.txt |   20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

docs/manual/configure.txt +8 -9

index 89df3d9..9b8135a 100644
@@ -216,7 +216,7 @@ x86_64 target, you have to generate a cross-compilation toolchain with
 Buildroot or crosstool-NG.
 
 If you want to generate a custom toolchain for your project, that can
-be used as an external toolchain in Buildroot, our recommandation is
+be used as an external toolchain in Buildroot, our recommendation is
 definitely to build it with http://crosstool-ng.org[crosstool-NG]. We
 recommend to build the toolchain separately from Buildroot, and then
 _import_ it in Buildroot using the external toolchain backend.

@@ -310,22 +310,24 @@ different solutions to handle the +/dev+ directory :
    about +mdev+ and the syntax of its configuration file, see
    http://git.busybox.net/busybox/tree/docs/mdev.txt.
 
- * The fourth solution is *Dynamic using udev*. This method also
+ * The fourth solution is *Dynamic using eudev*. This method also
    relies on the _devtmpfs_ virtual filesystem detailed above, but
-   adds the +udev+ userspace daemon on top of it. +udev+ is a daemon
+   adds the +eudev+ userspace daemon on top of it. +eudev+ is a daemon
    that runs in the background, and gets called by the kernel when a
    device gets added or removed from the system. It is a more
-   heavyweight solution than +mdev+, but provides higher flexibility
-   and is sometimes mandatory for some system components (systemd for
-   example). +udev+ is the mechanism used in most desktop Linux
-   distributions. For more details about +udev+, see
-   http://en.wikipedia.org/wiki/Udev.
+   heavyweight solution than +mdev+, but provides higher flexibility.
+   +eudev+ is a standalone version of +udev+, the original userspace
+   daemon used in most desktop Linux distributions, which is now part
+   of Systemd. For more details, see http://en.wikipedia.org/wiki/Udev.
 
-The Buildroot developers recommandation is to start with the *Dynamic
+The Buildroot developers recommendation is to start with the *Dynamic

Quoted 48 lines · Eric Le Bihan

This patch updates the /dev management section in the manual with information about eudev, which replaces udev.

Signed-off-by Eric Le Bihan <[email protected]>

---
 docs/manual/configure.txt |   20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

docs/manual/configure.txt +8 -9

index 89df3d9..9b8135a 100644
@@ -216,7 +216,7 @@ x86_64 target, you have to generate a cross-compilation toolchain with
 Buildroot or crosstool-NG.
 
 If you want to generate a custom toolchain for your project, that can
-be used as an external toolchain in Buildroot, our recommandation is
+be used as an external toolchain in Buildroot, our recommendation is
 definitely to build it with http://crosstool-ng.org[crosstool-NG]. We
 recommend to build the toolchain separately from Buildroot, and then
 _import_ it in Buildroot using the external toolchain backend.

@@ -310,22 +310,24 @@ different solutions to handle the +/dev+ directory :
    about +mdev+ and the syntax of its configuration file, see
    http://git.busybox.net/busybox/tree/docs/mdev.txt.
 
- * The fourth solution is *Dynamic using udev*. This method also
+ * The fourth solution is *Dynamic using eudev*. This method also
    relies on the _devtmpfs_ virtual filesystem detailed above, but
-   adds the +udev+ userspace daemon on top of it. +udev+ is a daemon
+   adds the +eudev+ userspace daemon on top of it. +eudev+ is a daemon
    that runs in the background, and gets called by the kernel when a
    device gets added or removed from the system. It is a more
-   heavyweight solution than +mdev+, but provides higher flexibility
-   and is sometimes mandatory for some system components (systemd for
-   example). +udev+ is the mechanism used in most desktop Linux
-   distributions. For more details about +udev+, see
-   http://en.wikipedia.org/wiki/Udev.
+   heavyweight solution than +mdev+, but provides higher flexibility.
+   +eudev+ is a standalone version of +udev+, the original userspace
+   daemon used in most desktop Linux distributions, which is now part
+   of Systemd. For more details, see http://en.wikipedia.org/wiki/Udev.
 
-The Buildroot developers recommandation is to start with the *Dynamic
+The Buildroot developers recommendation is to start with the *Dynamic
 using devtmpfs only* solution, until you have the need for userspace
 to be notified when devices are added/removed, or if firmwares are
 needed, in which case *Dynamic using mdev* is usually a good solution.
 

docs/manual/configure.txt different solutions to handle the +/dev+ directory : · line 333

Quoted 6 lines

docs/manual/configure.txt +3 -0

+Note that if +systemd+ is chosen as init system, /dev management will
+be performed by the +udev+ program provided by +systemd+.
+
 init system
 ~~~~~~~~~~~
 

Acked-by Thomas De Schampheleire <[email protected]>

This series converts udev to a virtual package, either provided by systemd or eudev.

Starting with version 183, udev has been merged into systemd. This forces the use of systemd as the init system if /dev is to be dynamically managed by udev. eudev is a fork of udev, maintained by Gentoo, but isolated from systemd, so it can be used with any init system.

Systemd has been bumped to v207 and a new eudev package has been added. Version 1.3 of eudev is in sync with systemd v207.

The bump of systemd to v207 also introduces new options, like the activation of the journal gateway.

In the end:

- if systemd is selected as init system, the /dev management will handled by udev. No other choice possible.
- if busybox or SysV is chosen, the user can choose to manage /dev statically, or dynamically using mdev or eudev.

Quoted 14 lines · Eric Le Bihan

This series converts udev to a virtual package, either provided by systemd or eudev.

Starting with version 183, udev has been merged into systemd. This forces the use of systemd as the init system if /dev is to be dynamically managed by udev. eudev is a fork of udev, maintained by Gentoo, but isolated from systemd, so it can be used with any init system.

Systemd has been bumped to v207 and a new eudev package has been added. Version 1.3 of eudev is in sync with systemd v207.

The bump of systemd to v207 also introduces new options, like the activation of the journal gateway.

Quoted 6 lines

In the end:

- if systemd is selected as init system, the /dev management will handled by udev. No other choice possible.
- if busybox or SysV is chosen, the user can choose to manage /dev statically, or dynamically using mdev or eudev.

Hi Eric,

I'm very happy with this patch series, it really cleans things up a lot.
I first had some reservations about how the choices were organised in the menus, but after trying out a few things it turns out to work pretty well.

I do agree with the comments Thomas has made, though. However, I
wouldn't want to stop this series to go into 2014.02-rc1, so I propose that Peter still merges it for the release and that these small things are fixed in follow-up patches.

I have one more remark about the way the udev virtual package is
implemented. We found out for the opengl packages that there is a better way to do it, something like:

udev/Config.in:

config BR2_PACKAGE_HAS_UDEV
bool

config BR2_PACKAGE_PROVIDES_UDEV
depends on BR2_PACKAGE_HAS_UDEV
string

udev/udev.mk:

UDEV_SRC =
UDEV_DEPENDENCIES = $(call qstrip,$(BR2_PACKAGE_PROVIDES_UDEV))

ifeq ($(UDEV_DEPENDENCIES),)
define UDEV_CONFIGURE_CMDS
echo "No Udev implementation selected. Configuration error." exit 1
endef
endif

eudev/Config.in:

...
if BR2_PACKAGE_EUDEV
config BR2_PACKAGE_PROVIDES_UDEV
default "eudev"
endif

That moves all the provider logic to the provider itself, which for
instance makes it possible for BR2_EXTERNAL to provide an alternative udev implementation (e.g. a different udev version, which could be _very_ relevant).

I repeat, though: this can be done as a follow-up patch.

Yes, the word 'provides' is more explicit. I'll put this in a follow-up patch. If this is the definitive way of implementing a virtual package, shouldn't it be added to docs/manual/adding-packages-tips.txt?

Quoted 1 line

Quoted 1 line

Quoted 1 line

Quoted 1 line

"Eric" == Eric Le Bihan <[email protected]> writes:

This series converts udev to a virtual package, either provided by systemd or eudev.

Starting with version 183, udev has been merged into systemd. This forces the use of systemd as the init system if /dev is to be dynamically managed by udev. eudev is a fork of udev, maintained by Gentoo, but isolated from systemd, so it can be used with any init system.

Systemd has been bumped to v207 and a new eudev package has been added. Version 1.3 of eudev is in sync with systemd v207.

The bump of systemd to v207 also introduces new options, like the activation of the journal gateway.

In the end:

- if systemd is selected as init system, the /dev management will handled by udev. No other choice possible.
- if busybox or SysV is chosen, the user can choose to manage /dev statically, or dynamically using mdev or eudev.

Changes v6 -> v7:
- updated udev related symbols and improved help text in Config.in.legacy (suggested by ThomasDS).
- removed redundant comments on dependencies for systemd and eudev (suggested by ThomasDS).
- fixed packages not properly converted to BR2_PACKAGE_HAS_UDEV (reported by SamuelM).