Re: Bug#653053: ldap2zone: Sending email every hour fill up the mail spool
On Sun, Dec 25, 2011 at 10:16:24 +0100, Petter Reinholdtsen wrote:
> diff -u ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
> --- ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
> +++ ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
> @@ -27,7 +27,7 @@
> if [ $? -ne 0 ]; then
> - echo -e "Reloading the zone '$domain' failed:\n$result" 1>&2
> -+ printf "Reloading the zone '$domain' failed: $result\n" 1>&2
> ++ logger -t ldap2bind "Reloading the zone '$domain' failed: $result\n" 1>&2
> + else
> -+ printf "Reloading the zone '$domain' was successful\n" 1>&2
> ++ logger -t ldap2bind "Reloading the zone '$domain' was successful\n" 1>&2
> fi
> done
Can't you just drop the else branch, and leave the failure case (i.e.
the one where there's something to say) alone?
Cheers,
Julien
Reply to: