From: Jan Baier <redacted>
Date: Wed, 18 Aug 2021 12:20:04 +0000 (+0200)
Subject: banhosts: Call killall with the -s
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=89dfd418989f3d1fb39dbe924b88a8b59ac3534d;p=openwrt-packages.git

banhosts: Call killall with the -s

Some versions of killall do support the `killall -SIGNAL` syntax and
have only `-s SIGNAL` which should be supported everywhere.

I see the problem with *killall (PSmisc) 23.3* on latest TurrisOS 5.2

Signed-off-by: Jan Baier <redacted>
---

diff --git a/utils/banhosts/Makefile b/utils/banhosts/Makefile
index 0087a6ff1..47a0ddbea 100644
--- a/utils/banhosts/Makefile
+++ b/utils/banhosts/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=banhostlist
 PKG_VERSION:=1
-PKG_RELEASE:=1
+PKG_RELEASE:=2
 
 PKG_BUILD_DIR=$(BUILD_DIR)/$(PKG_NAME)-$(PKG_VERSION)
 PKG_MAINTAINER:=Matteo Croce <matteo@openwrt.org>
diff --git a/utils/banhosts/files/updatebanhost b/utils/banhosts/files/updatebanhost
index a00d9cb6b..3930b60ad 100644
--- a/utils/banhosts/files/updatebanhost
+++ b/utils/banhosts/files/updatebanhost
@@ -6,7 +6,7 @@ gethosts() {
 	logger -t "banhost[$$]" "Update $file"
 	wget -qO- http://winhelp2002.mvps.org/hosts.txt |awk 'BEGIN{printf "0.0.0.0"}/^0\.0\.0\.0/{printf " "$2}END{exit(!FNR)}' >$file || exit 1
 	echo -n $time >$file.time
-	exec killall -HUP dnsmasq
+	exec killall -s HUP dnsmasq
 }
 
 if [ "$ACTION" = ifup -a "$INTERFACE" = wan ]; then