From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Rework of shutdown policy in inittab
Date: Wed, 14 Dec 2011 17:46:53 +0100 [thread overview]
Message-ID: <20111214174653.7319339e@skate> (raw)
In-Reply-To: <CAAXf6LU1QuFL=pOD09e0dzA1ntsauaFSD2qUr-OPEbmXB74fyw@mail.gmail.com>
Le Wed, 7 Dec 2011 07:52:20 +0100,
Thomas De Schampheleire <patrickdepinguin+buildroot@gmail.com> a ?crit :
> I think there are at least two solutions:
> 1. Keep the new behavior created with this patch, but make sure that
> the release notes clearly state this behavior change and update the
> documentation.
I'd go for this one. All initscripts in Buildroot itself implement the
start/stop/restart logic, so author of scripts that haven't been
submitted upstream should have read this as a suggestion of following a
similar scheme.
> 2. Adapt the new behavior: currently you are following
> /etc/init.d/Sxxxx files/links, while I think that the default policy
> on many distributions is to use /etc/init.d/Kxxxx for shutdown (and
> Sxxxx only for startup). In this scenario, users who have custom Sxxx
> init scripts will not be impacted. Only if they choose to, can they
> create appropriate Kxxxx links and provide stop functions.
> For scripts present in the buildroot distribution, we can provide
> appropriate S and K links immediately.
IMO, it would be ugly to have Sxxfoobar be a regular file (the script
itself) and Kxxfoobar be a symbolic link. So it means that we would
have to change all all scripts to /etc/init.d/foobar, and then create
symbolic links Sxxfoobar and Kxxfoobar. That's an alternative solution,
certainly, but it triples the number of files/symlinks required.
Regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2011-12-14 16:46 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-02 12:25 [Buildroot] [PATCH] Rework of shutdown policy in inittab Maxime Ripard
2011-11-24 8:32 ` Maxime Ripard
2011-12-07 4:31 ` Kelvin Cheung
2011-12-07 6:52 ` Thomas De Schampheleire
2011-12-07 7:26 ` Kelvin Cheung
2011-12-07 9:50 ` Gustavo Zacarias
2011-12-07 13:08 ` Maxime Ripard
2011-12-08 9:33 ` Thomas De Schampheleire
2011-12-14 16:46 ` Thomas Petazzoni [this message]
2011-12-15 8:44 ` Thomas De Schampheleire
2012-01-03 10:22 ` Maxime Ripard
2012-01-03 10:48 ` Thomas De Schampheleire
-- strict thread matches above, loose matches on Subject: below --
2011-12-15 13:21 Sagaert Johan
2011-12-15 15:08 ` Thomas De Schampheleire
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=20111214174653.7319339e@skate \
--to=thomas.petazzoni@free-electrons.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.