From: Eric Le Bihan <eric.le.bihan.dev@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package: udev is now provided by systemd or eudev.
Date: Tue, 17 Sep 2013 14:53:52 +0200 [thread overview]
Message-ID: <20130917125352.GB13723@pc-eric> (raw)
In-Reply-To: <20130917071737.4af20d60@skate>
On Tue, Sep 17, 2013 at 07:17:37AM +0200, Thomas Petazzoni wrote:
> (1) All the existing BR2_PACKAGE_UDEV* options are being renamed,
> without them being added in Config.in.legacy to provide a smooth
> migration path for users upgrading Buildroot. I believe this should be
> fixed by migrating them to eudev automatically through
> Config.in.legacy logic.
I was not aware of this mechanism, as I was previously working with an old
version of Buildroot (2012.08).
>
> (2) I'm not sure I like the fact that the selection of eudev vs.
> systemd to be "hidden" within the udev virtual package Config.in
> option. It means that the user needs to go in Target packages -> ...
> -> udev to be able to select between eudev vs. systemd. I believe this
> choice should be part of some option within the more global 'System
> configuration' menu, or derived from the existing options.
>
> There are I believe a few possibilities. The first one is to decide
> automatically based on the /dev management solution + the init system
> being used. If the /dev management solution is udev and the init
> system is *not* systemd, then eudev can be used automatically, while
> if the /dev management solution is udev and the init system is
> systemd, then systemd should be used.
>
> Another option is to explicitly create another /dev management option:
>
> - Static /dev
> - /dev with devtmpfs
> - /dev with devtmpfs + mdev
> - /dev with devtmpfs + eudev
> - /dev with devtmpfs + systemd/udev
>
> I don't have a particular preference at the moment, but I reallly
> believe it should be handled as a global System configuration option
> rather than as a udev virtual package option.
As mentioned in my reply to Arnout, I think it is better to have the selection
of the init system before the selection of the /dev management, because the
first item to choose when building a system is how to start it, not how to
manage the devices.
Our constraints are:
1) we need to keep a virtual udev package to avoid breaking Pulseaudio, Weston,
Udisks, etc: as eudev is identical to udev from systemd, it does not
matter which one is selected to build these packages.
2) there is no need to offer a static /dev, mdev or eudev if systemd is
selected. The opposite is allowed.
I ack the idea of moving the choice between udev (from systemd) and eudev to
the "System Configuration" menu. So I suggest that, when the user selects
the init system:
- if systemd is selected, then the menu to select /dev management is *not*
displayed, but replaced by a comment as "/dev management with udev
(from systemd)". The virtual package udev is selected.
- if systemd is not selected, then the menu to select /dev management is
displayed, with the following choices:
* static /dev
* /dev with devtmpfs
* /dev with devtmpfs + mdev
* /dev with devtmpfs + eudev
Selecting eudev selects the udev virtual package. The user can configure
the extras for eudev in the appropriate "Target packages" section.
My reply to Arnout should contain the answers to the other questions.
next prev parent reply other threads:[~2013-09-17 12:53 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-06 14:01 [Buildroot] [PATCH 1/1] package: udev is now provided by systemd or eudev eric.le.bihan.dev at free.fr
2013-09-06 14:08 ` Eric Le Bihan
2013-09-10 20:24 ` Arnout Vandecappelle
2013-09-17 10:40 ` Eric Le Bihan
2013-09-18 4:45 ` Thomas Petazzoni
2013-09-18 6:52 ` Arnout Vandecappelle
2013-09-17 18:37 ` Thomas Petazzoni
2013-09-18 6:57 ` Arnout Vandecappelle
2013-09-18 15:56 ` Thomas Petazzoni
2013-09-18 16:04 ` Arnout Vandecappelle
2013-09-18 16:40 ` Thomas Petazzoni
2013-09-18 21:46 ` Arnout Vandecappelle
2013-09-17 5:17 ` Thomas Petazzoni
2013-09-17 12:53 ` Eric Le Bihan [this message]
2013-09-17 18:45 ` Thomas Petazzoni
2013-09-18 7:00 ` Arnout Vandecappelle
2013-09-18 15:58 ` Thomas Petazzoni
2013-09-18 16:06 ` Arnout Vandecappelle
2013-09-18 16:41 ` Thomas Petazzoni
2013-09-18 17:34 ` Sagaert Johan
2013-09-18 17:39 ` Thomas Petazzoni
2013-09-18 18:13 ` Sagaert Johan
2013-09-18 18:05 ` [Buildroot] [PATCH 1/1] package: udev is now provided bysystemd " Sagaert Johan
2013-09-18 18:16 ` Thomas Petazzoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130917125352.GB13723@pc-eric \
--to=eric.le.bihan.dev@free.fr \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox