From 6b0ee8b283948e09510ed439e999ded1e164b1c9 Mon Sep 17 00:00:00 2001
From: Stuart Cardall <developer@it-offshore.co.uk>
Date: Tue, 28 Jan 2020 10:23:19 +0000
Subject: [PATCH] add pidof to required depends

Centos may not have sysvinit-tools installed by default. Adds a check
for pidof being available.

Fixes:

https://github.com/mitchellkrogza/nginx-ultimate-bad-bot-blocker/issues/321#issuecomment-578514104
---
 install-ngxblocker | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/install-ngxblocker b/install-ngxblocker
index af6479182..2f58d7781 100755
--- a/install-ngxblocker
+++ b/install-ngxblocker
@@ -258,13 +258,20 @@ find_binary() {
 
 check_depends() {
 	# some distros do not have these tools installed by default
-	if [ -z $(find_binary wget) ]; then
-		printf "$0 requires: 'wget' \n"
-		exit 1
-	fi
+	local x= depends_list="wget curl"
 
-	if [ -z $(find_binary curl) ]; then
-		printf "$0 requires 'curl' \n"
+	for x in $depends_list; do
+		if [ -z $(find_binary $x) ]; then
+			printf "$0 requires: '$x' \n"
+			exit 1
+		fi
+	done
+
+	# give a helpful message for missing pidof
+	if [ -z $(find_binary pidof) ]; then
+		printf "$0 requires 'pidof' \n\n"
+		printf "In Debian: apt install sysvinit-utils\n"
+		printf "In Centos: yum install sysvinit-tools\n"
 		exit 1
 	fi
 }