From 3675b489e2576ab36ac6c61cf4b8f16d34f8da5b Mon Sep 17 00:00:00 2001
From: tv <tv@shackspace.de>
Date: Mon, 11 Aug 2014 22:01:41 +0200
Subject: Reaktor nag: dry up output function with eval^_^

---
 commands/nag | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

(limited to 'commands/nag')

diff --git a/commands/nag b/commands/nag
index fabeec8..8962d9f 100755
--- a/commands/nag
+++ b/commands/nag
@@ -40,26 +40,28 @@ if test "${inspect_services-false}" = true; then
   missing_mailtos=$(set +f; cd nag.services && grep '^mailto:$' * | cut -d: -f1)
 fi
 
+nag_missing_services='missing services'
+nag_obsolete_services='obsolete services'
+nag_missing_owners='services with missing owner entry'
+nag_missing_mailtos='services with missing mailto entry'
+
 #
 # output
 #
 did_nag=false
-if test -n "$missing_services"; then
-  echo missing services: $missing_services
-  did_nag=true
-fi
-if test -n "$obsolete_services"; then
-  echo obsolete services: $obsolete_services
-  did_nag=true
-fi
-if test -n "${missing_owners-}"; then
-  echo services with missing owner entry: $missing_owners
-  did_nag=true
-fi
-if test -n "${missing_mailtos-}"; then
-  echo services with missing mailto entry: $missing_mailtos
-  did_nag=true
-fi
+for i in \
+    missing_services \
+    obsolete_services \
+    missing_owners \
+    missing_mailtos
+do
+  eval "offenders=\$$i; nag_msg=\$nag_$i"
+  if test -n "$offenders"; then
+    echo $nag_msg: $offenders
+    did_nag=true
+  fi
+done
+
 # if asked directly, answer if there is nothing to nag about
 if test $did_nag = false -a -n "${_from:-}";then
   echo "nothing to nag about"
-- 
cgit v1.2.3