[PATCH] squid init v3.1: Bugfix release

[Matthias Fischer <matthias.fischer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 2377 bytes --]

- Fixed and adjusted non-working 'while'-loop for stopping still running
  '(squid-1)'-processes.

- Commented 'wait' after 'killproc ...'-block: until now, no troubles found.

- Cosmetic tuning.

Best,
Matthias

Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
---
 src/initscripts/init.d/squid | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/src/initscripts/init.d/squid b/src/initscripts/init.d/squid
index 47bf182..e5c9b85 100644
--- a/src/initscripts/init.d/squid
+++ b/src/initscripts/init.d/squid
@@ -95,9 +95,9 @@ case "$1" in
 			iptables -t nat -F SQUID
 		if [ -e /var/run/squid.pid ]; then
 			boot_mesg -n "Shutting down Squid Proxy Server...\n"
-			boot_mesg "(this may take up to a few minutes)."
+			boot_mesg "(this may take up to a few minutes)"
 			/usr/sbin/squid -k shutdown >/dev/null 2>&1
-			evaluate_retval
+#			evaluate_retval
 
 			# Stop squidGuard, updxlrator, squidclamav
 			# and redirect_wrappers.
@@ -107,31 +107,38 @@ case "$1" in
 			killproc /usr/sbin/redirect_wrapper >/dev/null &
 
 			# Wait until all redirectors have been stopped.
-			wait
+#			wait
 
 			# If some squid processes are still running, wait up to 360 seconds
 			# before we go on to kill all and delete entire cache structure.
+
 			n=0
-			while [ /usr/sbin/squid -k check > /dev/null 2>&1 ] && [ $n -lt 360 ]; do
-				sleep 1
-				n=$(( ${n} + 1 ))
+			while /usr/sbin/squid -k check > /dev/null 2>&1 && [ $n -lt 360 ]; do
+				sleep 2
+				n=$(( ${n} + 2 ))
+				echo -n .
 			done
+			echo ""
+			echo "Done."
+			echo "Shutdown time:" "$n" "seconds"
 
 			# If (squid-1) is still running after 360 seconds,
 			# kill all squid processes and delete damaged cache structure.
 			if ( pgrep -fl "(squid-1)" > /dev/null 2>&1 ); then
 				killproc /usr/sbin/squid >/dev/null
 				rm -rf /var/log/cache/*
+				echo ""
 				boot_mesg -n "You should not be reading this warning.\n"
 				boot_mesg -n "Some squid-processes had to be killed after 360 seconds,\n"
 				boot_mesg -n "so index file and cache contents had to be deleted.\n"
 				boot_mesg -n "Therefore, the complete cache structure will be rebuild\n"
 				boot_mesg "during next start."
 				echo_warning
+				echo ""
 			else
 				boot_mesg "All squid processes exited normally."
 				echo_ok
-				boot_mesg ""
+				echo ""
 			fi
 		fi
 
-- 
2.8.2