adblock: update 3.8.9

* more startup tweaks
* re-use f_log function in helper scripts
* small fixes / polish up for forthcoming 19.07 release

Signed-off-by: Dirk Brenken <dev@brenken.org>

net/adblock/Makefile

@@ -6,7 +6,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=adblock
-PKG_VERSION:=3.8.8
+PKG_VERSION:=3.8.9
 PKG_RELEASE:=1
 PKG_LICENSE:=GPL-3.0-or-later
 PKG_MAINTAINER:=Dirk Brenken <dev@brenken.org>

net/adblock/files/adblock.init

@@ -15,14 +15,15 @@ adb_init="/etc/init.d/adblock"
 adb_script="/usr/bin/adblock.sh"
 adb_pidfile="/var/run/adblock.pid"
 
-if [ -s "${adb_pidfile}" ] && [ "${action}" != "help" ] && [ "${action}" != "status" ]
+if [ -s "${adb_pidfile}" ] && \
+	[ "${action}" != "help" ] && [ "${action}" != "status" ] && [ "${action}" != "boot" ]
 then
 	exit 1
 fi
 
 boot()
 {
-	adb_boot=1
+	[ -s "${adb_pidfile}" ] && > "${adb_pidfile}"
 	rc_procd start_service
 }
 
@@ -30,19 +31,17 @@ start_service()
 {
 	if [ "$("${adb_init}" enabled; printf "%u" ${?})" -eq 0 ]
 	then
-		if [ -n "${adb_boot}" ]
+		if [ "${action}" = "boot" ]
 		then
-			local trigger="$(uci_get adblock global adb_trigger)"
+			if [ "$(uci_get adblock global adb_trigger)" != "timed" ]
-			if [ "${trigger}" != "timed" ]
 			then
 				return 0
 			fi
 		fi
-		local nice="$(uci_get adblock extra adb_nice "0")"
 		procd_open_instance "adblock"
 		procd_set_param command "${adb_script}" "${@}"
 		procd_set_param pidfile "${adb_pidfile}"
-		procd_set_param nice "${nice}"
+		procd_set_param nice "$(uci_get adblock extra adb_nice "0")"
 		procd_set_param stdout 1
 		procd_set_param stderr 1
 		procd_close_instance

net/adblock/files/adblock.mail

@@ -25,6 +25,24 @@ adb_logger="$(command -v logger)"
 adb_logread="$(command -v logread)"
 adb_rc=1
 
+f_log()
+{
+	local class="${1}" log_msg="${2}"
+
+	if [ -x "${adb_logger}" ]
+	then
+		"${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+	else
+		printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
+	fi
+}
+
+if [ -z "${adb_mreceiver}" ]
+then
+	f_log "err" "please set the mail receiver with the 'adb_mreceiver' option"
+	exit ${adb_rc}
+fi
+
 if [ "${adb_debug}" -eq 1 ]
 then
 	debug="--debug"
@@ -58,20 +76,8 @@ if [ -x "${adb_mail}" ] && [ -n "${adb_mreceiver}" ]
 then
 	printf "%b" "${adb_mhead}${adb_mtext}" 2>/dev/null | "${adb_mail}" ${debug} -a "${adb_mprofile}" "${adb_mreceiver}" >/dev/null 2>&1
 	adb_rc=${?}
-	class="info"
+	f_log "info" "mail sent to '${adb_mreceiver}' with rc '${adb_rc}'"
-	log_msg="mail sent to '${adb_mreceiver}' with rc '${adb_rc}'"
-elif [ -z "${adb_mreceiver}" ]
-then
-	class="err"
-	log_msg="please set the mail receiver with the 'adb_mreceiver' option"
 else
-	class="err"
+	f_log "err" "msmtp mail daemon not found"
-	log_msg="msmtp mail daemon not found"
-fi
-if [ -x "${adb_logger}" ]
-then
-	"${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
-else
-	printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
 fi
 exit ${adb_rc}

net/adblock/files/adblock.service

@@ -18,20 +18,23 @@ adb_ver="${1}"
 adb_ubus="$(command -v ubus)"
 adb_logger="$(command -v logger)"
 
+f_log()
+{
+	local class="${1}" log_msg="${2}"
+
+	if [ -x "${adb_logger}" ]
+	then
+		"${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
+	else
+		printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
+	fi
+}
+
 if [ -x "${adb_ubus}" ] && [ -n "${adb_dns}" ]
 then
+	f_log "info" "ubus/adblock service started"
 	"${adb_ubus}" -S -M r -m invoke monitor | \
 		{ grep -qE "\"method\":\"(set|signal)\",\"data\":\{\"name\":\"${adb_dns}\""; [ $? -eq 0 ] && /etc/init.d/adblock start; }
-	class="info"
-	log_msg="ubus/adblock service started"
 else
-	class="err"
+	f_log "err" "can't start ubus/adblock service"
-	log_msg="can't start ubus/adblock service"
 fi
-if [ -x "${adb_logger}" ]
-then
-	"${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
-else
-	printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
-fi
-

net/adblock/files/adblock.sh

@@ -13,7 +13,7 @@
 #
 LC_ALL=C
 PATH="/usr/sbin:/usr/bin:/sbin:/bin"
-adb_ver="3.8.8"
+adb_ver="3.8.9"
 adb_basever=""
 adb_enabled=0
 adb_debug=0
@@ -283,13 +283,12 @@ f_temp()
 	if [ -d "/tmp" ] && [ -z "${adb_tmpdir}" ]
 	then
 		adb_tmpdir="$(mktemp -p /tmp -d)"
-		adb_tmpload="$(mktemp -p "${adb_tmpdir}" -tu)"
 		adb_tmpfile="$(mktemp -p "${adb_tmpdir}" -tu)"
 	elif [ ! -d "/tmp" ]
 	then
 		f_log "err" "the temp directory '/tmp' does not exist/is not mounted yet, please create the directory or raise the 'adb_triggerdelay' to defer the adblock start"
 	fi
-	if [ ! -s "${adb_pidfile}" ]
+	if [ ! -f "${adb_pidfile}" ] || [ ! -s "${adb_pidfile}" ]
 	then
 		printf "%s" "${$}" > "${adb_pidfile}"
 	fi
@@ -929,13 +928,6 @@ f_log()
 			f_rmdns
 			f_bgserv "stop"
 			f_jsnup "error"
-			log_msg="Please also check 'https://github.com/openwrt/packages/blob/master/net/adblock/files/README.md'"
-			if [ -x "${adb_logger}" ]
-			then
-				"${adb_logger}" -p "${class}" -t "adblock-${adb_ver}[${$}]" "${log_msg}"
-			else
-				printf "%s %s %s\\n" "${class}" "adblock-${adb_ver}[${$}]" "${log_msg}"
-			fi
 			exit 1
 		fi
 	fi
@@ -979,8 +971,8 @@ f_main()
 		src_url="$(eval printf "%s" \"\$\{adb_src_${src_name}\}\")"
 		src_rset="$(eval printf "%s" \"\$\{adb_src_rset_${src_name}\}\")"
 		src_cat="$(eval printf "%s" \"\$\{adb_src_cat_${src_name}\}\")"
-		src_tmpload="${adb_tmpload}.${src_name}"
+		src_tmpload="${adb_tmpfile}.${src_name}.load"
-		src_tmpfile="${adb_tmpfile}.${src_name}"
+		src_tmpfile="${adb_tmpfile}.${src_name}.file"
 		src_rc=4
 
 		# basic pre-checks