commit:     c333707cba356f4cacfd58a6fcc78f7c073dddcd
Author:     William Hubbs <w.d.hubbs <AT> gmail <DOT> com>
AuthorDate: Tue Mar 14 23:04:31 2017 +0000
Commit:     William Hubbs <williamh <AT> gentoo <DOT> org>
CommitDate: Tue Mar 14 23:04:31 2017 +0000
URL:        https://gitweb.gentoo.org/proj/openrc.git/commit/?id=c333707c

Remove all occurances of 'before *' from dependencies

Using wildcards in dependencies causes issues when rc_parallel is set to
yes because it can lead to deadlocks.
All dependencies need to be explicit rather than implicit.

This is the first stage of moving this direction.

 init.d/hwclock.in | 3 ++-
 init.d/swclock.in | 5 ++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/init.d/hwclock.in b/init.d/hwclock.in
index f78cc7c3..3bfcab1d 100644
--- a/init.d/hwclock.in
+++ b/init.d/hwclock.in
@@ -34,7 +34,8 @@ depend()
        if yesno $clock_adjfile; then
                use root
        else
-               before *
+               before binfmt bootmisc fsck hostname keymaps localmount 
loopback mtab
+               before procfs root swap sysctl termencoding urandom
        fi
        keyword -docker -lxc -openvz -prefix -systemd-nspawn -uml -vserver -xenu
 }

diff --git a/init.d/swclock.in b/init.d/swclock.in
index 9ceba444..67baaab0 100644
--- a/init.d/swclock.in
+++ b/init.d/swclock.in
@@ -13,7 +13,10 @@ description="Sets the local clock to the mtime of a given 
file."
 
 depend()
 {
-       before *
+       before adjkerntz binfmt bootmisc devdb dumpon fsck hostid hostname 
keymaps
+       before localmount loopback modules mtab network newsyslog procfs root
+       before savecore staticroute swap swap-blk syscons sysctl syslogd
+       before termencoding ttys urandom wscons
        provide clock
        keyword -docker -lxc -openvz -prefix -systemd-nspawn -uml -vserver -xenu
 }

Reply via email to