From 3675b489e2576ab36ac6c61cf4b8f16d34f8da5b Mon Sep 17 00:00:00 2001 From: tv 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