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 -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20111225172426.gb7...@radis.cristau.org