mountnfs.sh uses "service rpcbind status" only to avoid
starting rpcbind twice before mounting remote filesystems. That
pulls init-system-helpers-service into images solely to provide
/usr/sbin/service.

Use start-stop-daemon test mode for the rpcbind status check
instead. The script already uses start-stop-daemon to start rpcbind,
so this keeps the duplicate-start protection without requiring the
Debian service wrapper.

With mountnfs.sh no longer invoking service, drop
init-system-helpers-service from the initscripts runtime dependencies.

Signed-off-by: Shinu Chandran <[email protected]>
Signed-off-by: Richard Purdie <[email protected]>
---
 meta/recipes-core/initscripts/initscripts-1.0/mountnfs.sh | 3 +--
 meta/recipes-core/initscripts/initscripts_1.0.bb          | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-core/initscripts/initscripts-1.0/mountnfs.sh 
b/meta/recipes-core/initscripts/initscripts-1.0/mountnfs.sh
index c08597e725..53c6cfd457 100755
--- a/meta/recipes-core/initscripts/initscripts-1.0/mountnfs.sh
+++ b/meta/recipes-core/initscripts/initscripts-1.0/mountnfs.sh
@@ -71,8 +71,7 @@ if test "$rpcbind" = yes
 then
        if test -x /usr/sbin/rpcbind
        then
-               service rpcbind status > /dev/null
-               if [ $? != 0 ]; then
+               if ! start-stop-daemon --stop --test --quiet --exec 
/usr/sbin/rpcbind; then
                        echo -n "Starting rpcbind..."
                        start-stop-daemon --start --quiet --exec 
/usr/sbin/rpcbind
                        sleep 2
diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb 
b/meta/recipes-core/initscripts/initscripts_1.0.bb
index 56ee65ac5b..fbaf7d51cd 100644
--- a/meta/recipes-core/initscripts/initscripts_1.0.bb
+++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
@@ -48,7 +48,6 @@ PACKAGE_WRITE_DEPS:append = " 
${@bb.utils.contains('DISTRO_FEATURES','systemd','
 PACKAGES =+ "${PN}-functions ${PN}-sushell"
 RDEPENDS:${PN} = "initd-functions \
                   
${@bb.utils.contains('DISTRO_FEATURES','selinux','${PN}-sushell','',d)} \
-                  init-system-helpers-service \
                 "
 # Recommend pn-functions so that it will be a preferred default provider for 
initd-functions
 RRECOMMENDS:${PN} = "${PN}-functions"
-- 
2.35.6

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#244191): 
https://lists.openembedded.org/g/openembedded-core/message/244191
Mute This Topic: https://lists.openembedded.org/mt/120916440/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

  • [OE-core] [scarthgap] [PATCH] in... Shinu Chandran via lists.openembedded.org

Reply via email to