rpcd-mod-*: improve postinst script 4646/head
authorKarel Kočí <karel.koci@nic.cz>
Mon, 7 Dec 2020 16:01:17 +0000 (17:01 +0100)
committerKarel Kočí <karel.koci@nic.cz>
Mon, 7 Dec 2020 16:01:17 +0000 (17:01 +0100)
Usage of killall is replaced with init script. This is cleaner solution
as it does not consider some implementation detail but rather passes
that on to init script implementation.

IPKG_INSTROOT was added to prevent execution when not running in current
root. It is invalid to request reload if install-root is not current
root. In this case it can be considered harmless but it is invalid
nonetheless.

Last change is removal of `exit 0`. This caused skip of default
postinst. Execution of default postinst does no harm and is more
standard considering possible future expansion of it.

Signed-off-by: Karel Kočí <karel.koci@nic.cz>
libs/rpcd-mod-luci/Makefile
libs/rpcd-mod-rad2-enc/Makefile
libs/rpcd-mod-rrdns/Makefile

index 8833bb45ca1f30f7ca47c31625c1aec591ead12e..a39ec3b2242152672a44f5c095381465dedacab9 100644 (file)
@@ -48,8 +48,7 @@ endef
 
 define Package/rpcd-mod-luci/postinst
 #!/bin/sh
-killall -HUP rpcd 2>/dev/null
-exit 0
+[ -n "$$IPKG_INSTROOT" ] || /etc/init.d/rpcd reload
 endef
 
 $(eval $(call BuildPackage,rpcd-mod-luci))
index 121e3a274feb5b411083d2e2f2b50a77547806a5..1dd61476aa72cc08038e25a0c2f052a28bea1f85 100644 (file)
@@ -41,8 +41,7 @@ endef
 
 define Package/rpcd-mod-rad2-enc/postinst
 #!/bin/sh
-killall -HUP rpcd 2>/dev/null
-exit 0
+[ -n "$$IPKG_INSTROOT" ] || /etc/init.d/rpcd reload
 endef
 
 $(eval $(call BuildPackage,rpcd-mod-rad2-enc))
index f0bf140a878e2aff29ae79b0f525832860abe03e..37f0e04466a8a8da35033f701afd5bde9e782c49 100644 (file)
@@ -40,8 +40,7 @@ endef
 
 define Package/rpcd-mod-rrdns/postinst
 #!/bin/sh
-killall -HUP rpcd 2>/dev/null
-exit 0
+[ -n "$$IPKG_INSTROOT" ] || /etc/init.d/rpcd reload
 endef
 
 $(eval $(call BuildPackage,rpcd-mod-rrdns))