From 0a88e3a38f5c0bb2ede4ceef1989a74422d75224 Mon Sep 17 00:00:00 2001
From: Mitchell Krog <mitchellkrog@gmail.com>
Date: Wed, 20 Sep 2017 12:15:50 +0200
Subject: [PATCH] Re-Test

---
 install-ngxblocker | 15 +++++++++++++++
 setup-ngxblocker   |  5 +++--
 update-ngxblocker  |  4 ++--
 3 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/install-ngxblocker b/install-ngxblocker
index 4ebb34a12..2ca0f621c 100755
--- a/install-ngxblocker
+++ b/install-ngxblocker
@@ -141,6 +141,16 @@ download_files() {
 	fi
 }
 
+set_mode() {
+	local mode=$1 dir=$2 file=
+	local file_list="$(echo $@ | awk '{$1=$2=""; print}' | sed -e 's/^[ \t]*//')"
+
+	for file in $file_list; do
+		printf "Setting mode: $mode => $dir/$file\n"
+		chmod $mode $dir/$file
+	done
+}
+
 check_config() {
 	local x= dirs="$*"
 
@@ -283,6 +293,11 @@ main() {
 	download_files conf.d $CONF_DIR $CONF_FILES
 	download_files bots.d $BOTS_DIR $BOT_FILES
 	download_files / $SCRIPT_DIR $SCRIPT_FILES
+
+	# ensures scripts are executable
+	if [ "$DRY_RUN" = "N" ]; then
+		set_mode 700 $SCRIPT_DIR $SCRIPT_FILES
+	fi
 }
 
 ## START ##
diff --git a/setup-ngxblocker b/setup-ngxblocker
index c84e5527f..30f24d79a 100755
--- a/setup-ngxblocker
+++ b/setup-ngxblocker
@@ -186,8 +186,9 @@ find_line() {
 	local file=$1 find_str=$2 first_last=$3
 
 	case "$first_last" in
-		first) awk "/$find_str/{ print NR; exit }" $file;;
-		 last) awk "/$find_str/{ print NR }" $file | tail -n1;;
+		# ignore file #comments
+		first) grep -nE "^[ ]+$find_str" $file | head -n1 | awk -F: '{print $1}'
+		 last) grep -nE "^[ ]+$find_str" $file | tail -n1 | awk -F: '{print $1}'
 	esac
 }
 
diff --git a/update-ngxblocker b/update-ngxblocker
index 186a513d7..b3869e5da 100755
--- a/update-ngxblocker
+++ b/update-ngxblocker
@@ -194,8 +194,8 @@ check_args() {
 }
 
 check_mail_depends() {
-	if [ ! -f /usr/bin/mail ]; then # mailx + ssmtp are enough to send emails
-		printf "${BOLDYELLOW}WARN${RESET}: missing /usr/bin/mail => ${BOLDWHITE}disabling emails${RESET}.\n\n"
+	if [ ! -f /usr/bin/mail ] && [ ! -f /bin/mail ]; then # mailx + ssmtp are enough to send emails
+		printf "${BOLDYELLOW}WARN${RESET}: missing mail command => ${BOLDWHITE}disabling emails${RESET}.\n\n"
 		SEND_EMAIL="N"
 	fi
 }