Re: Bug#653053: ldap2zone: Sending email every hour fill up the mail spool

[Holger Levsen <holger@layer-acht.org>]

Hi,

so I have this (also attached), it builds + works fine, may I upload?

$ debdiff ldap2zone_0.1-7.dsc ldap2zone_0.1-7.1.dsc
diff -u ldap2zone-0.1/debian/changelog ldap2zone-0.1/debian/changelog
--- ldap2zone-0.1/debian/changelog
+++ ldap2zone-0.1/debian/changelog
@@ -1,3 +1,13 @@
+ldap2zone (0.1-7.1) stable-proposed-updates; urgency=low
+
+  * Non-maintainer upload.
+  * Include half the patch from #653053 to silence the cronjob on success, to
+    avoid excessive emails and instead log this with syslog.
+    The full patch has been accepted by the maintainer and is already in 
+    wheezy and sid in version 0.2-2. (Closes: #653053)
+
+ -- Holger Levsen <holger@debian.org>  Thu, 05 Jan 2012 15:59:49 +0100
+
 ldap2zone (0.1-7) unstable; urgency=low
 
   * Corrected error in notify dns statement breaking the program
diff -u ldap2zone-0.1/debian/patches/00list 
ldap2zone-0.1/debian/patches/00list
--- ldap2zone-0.1/debian/patches/00list
+++ ldap2zone-0.1/debian/patches/00list
@@ -9,0 +10 @@
+11_be_silent_on_success_653053
only in patch2:
unchanged:
--- ldap2zone-0.1.orig/debian/patches/11_be_silent_on_success_653053.dpatch
+++ ldap2zone-0.1/debian/patches/11_be_silent_on_success_653053.dpatch
@@ -0,0 +1,21 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 11_be_silent_on_success_653053.dpatch by Holger Levsen <holger@debian.org>
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: Made script log using logger by a patch contributed by
+## DP: Petter Reinholdtsen
+## DP: modified to only do this on success for squeeze by Holger Levsen
+## DP: see #653053
+
+@DPATCH@
+
+
+--- ldap2zone-0.2.orig/ldap2bind
++++ ldap2zone-0.2/ldap2bind
+@@ -85,6 +85,6 @@
+               printf "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"
+       fi
+ done


cheers,
	Holger

diff -u ldap2zone-0.1/debian/changelog ldap2zone-0.1/debian/changelog
--- ldap2zone-0.1/debian/changelog
+++ ldap2zone-0.1/debian/changelog
@@ -1,3 +1,13 @@
+ldap2zone (0.1-7.1) stable-proposed-updates; urgency=low
+
+  * Non-maintainer upload.
+  * Include half the patch from #653053 to silence the cronjob on success, to
+    avoid excessive emails and instead log this with syslog.
+    The full patch has been accepted by the maintainer and is already in 
+    wheezy and sid in version 0.2-2. (Closes: #653053)
+
+ -- Holger Levsen <holger@debian.org>  Thu, 05 Jan 2012 15:59:49 +0100
+
 ldap2zone (0.1-7) unstable; urgency=low
 
   * Corrected error in notify dns statement breaking the program
diff -u ldap2zone-0.1/debian/patches/00list ldap2zone-0.1/debian/patches/00list
--- ldap2zone-0.1/debian/patches/00list
+++ ldap2zone-0.1/debian/patches/00list
@@ -9,0 +10 @@
+11_be_silent_on_success_653053
only in patch2:
unchanged:
--- ldap2zone-0.1.orig/debian/patches/11_be_silent_on_success_653053.dpatch
+++ ldap2zone-0.1/debian/patches/11_be_silent_on_success_653053.dpatch
@@ -0,0 +1,21 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 11_be_silent_on_success_653053.dpatch by Holger Levsen <holger@debian.org>
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: Made script log using logger by a patch contributed by
+## DP: Petter Reinholdtsen
+## DP: modified to only do this on success for squeeze by Holger Levsen
+## DP: see #653053
+
+@DPATCH@
+
+
+--- ldap2zone-0.2.orig/ldap2bind
++++ ldap2zone-0.2/ldap2bind
+@@ -85,6 +85,6 @@
+		printf "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"
+ 	fi
+ done