Package: release.debian.org
Severity: normal
User: release.debian....@packages.debian.org
Usertags: unblock

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Hi Release-Team

Please unblock package couriergraph

I did a NMU upload for couriergraph.

The upload addresses two RC bugs: One was opened already by Thomas
Goirand <z...@debian.org>: #689884.

The binary package shipped /var/run. /var/run/servergraph is now
created in initscript.

#692157: It was previously possible to start multiple instances of the
couriergraph daemon. I tried to workaround this checking if pid file
is present and then don't start another instance.

Furhermore a Brazilian Portuguese debconf templates translation update
was added.

Would it be possible to unblock the package for wheezy?

Attached is the debdiff against the package in testing.

unblock couriergraph/0.25-4.2

Regards,
Salvatore

- -- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/8 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBCgAGBQJQmWxiAAoJEHidbwV/2GP+GSUP/3NPtVZ0m8AWEYJcWJgbxomf
VBCyRm5YZzzm6t4t1RH7l78XU5eEbec0zNkj1pZsAZdlrAYE9IccYGIF6JQSJQab
I5BJl1iEGgTY5/OCq5UVAtLZGmh3GrrsW1hJmIuEW6SljxlCWVhhx9gGYxaQ4D9a
HlqDVAvRt0H9vaCh2IyQo+/8Z64TdVHh5MZ8L0hq58GN77Aq7pdSNTcq2Z4duVaI
10FqTQGN7WvJmEjsQ50pRAp6O7kFDr5AdkZJdZ+vES8hdkXcrwXrbOu0rexKHwxU
MIuNxBI3hH/MiPLStEoiQkEiEuXuRWUUSWmZ4l0M5pswzfCbEDXmSxyPAIygWV71
7nu/Ld2Ts1qfAYxP+ouDlZjkj6O8rV3FHdjej1jVyBVJUmufRgN97nz3zSs21CLj
mIfmH+To8uBPepfFtzfSyS+6WAvgJdmooBIPe/Ze3zhUU7TE4ATxrCCk7Y3iEFFT
GrDrZ3S5N/2IPE8RmJCuuDFw/p9t4FqhhhLHj+UOdS25zqZPDFVqNeQQcniaxd3i
WDJQ1x9x6NXuZwoa/GasbvVnTnlfFJOUSw1FIGAuEz8NVkl+NiKYyjZqORYQSfwR
nxAj9+pSWcWGr3BwryLiQLVtydQQG01xoUpMANlbiGWMIuuaDz1bpeSNEsUZH+CM
F94oJLjpCl4m1ER7S0c6
=dyNC
-----END PGP SIGNATURE-----
Base version: couriergraph_0.25-4.1 from testing
Target version: couriergraph_0.25-4.2 from unstable

No hints in place.

 changelog   |   17 +++++++++++++++++
 dirs        |    1 -
 init        |   24 ++++++++++++++++++------
 po/pt_BR.po |   50 +++++++++++++++++++++-----------------------------
 postinst    |    4 ----
 5 files changed, 56 insertions(+), 40 deletions(-)

diff -u couriergraph-0.25/debian/changelog couriergraph-0.25/debian/changelog
--- couriergraph-0.25/debian/changelog
+++ couriergraph-0.25/debian/changelog
@@ -1,3 +1,20 @@
+couriergraph (0.25-4.2) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Don't provide /var/run/servergraph in binary package.
+    Fix "Ships a folder in /var/run or /var/lock (Policy Manual section
+    9.3.2)". Create /var/run/servergraph in initscript if
+    /var/run/servergraph is not present.
+    Thanks to Thomas Goirand <z...@debian.org> (Closes: #689884)
+  * Check if daemon pid file is present and don't start daemon.
+    Check if daemon pid is present. If pid file is present assume the daemon
+    is still running and do not start another instance.
+    This is otherwise a Debian Policy 9.3.2 violation. (Closes: #692157)
+  * Update Brazilian Portuguese debconf templates translation.
+    Thanks to Adriano Rafael Gomes <adrian...@gmail.com> (Closes: #662169)
+
+ -- Salvatore Bonaccorso <car...@debian.org>  Fri, 02 Nov 2012 21:22:11 +0100
+
 couriergraph (0.25-4.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff -u couriergraph-0.25/debian/dirs couriergraph-0.25/debian/dirs
--- couriergraph-0.25/debian/dirs
+++ couriergraph-0.25/debian/dirs
@@ -4 +3,0 @@
-var/run/servergraph
diff -u couriergraph-0.25/debian/init couriergraph-0.25/debian/init
--- couriergraph-0.25/debian/init
+++ couriergraph-0.25/debian/init
@@ -14,7 +14,8 @@
 DESC="Courier login Statistics"
 DAEMON=/usr/sbin/couriergraph.pl
 
-PIDFILE=/var/run/servergraph/couriergraph.pid
+PIDDIR=/var/run/servergraph
+PIDFILE=$PIDDIR/couriergraph.pid
 RRD_DIR=/var/lib/couriergraph
 RRD_NAME=couriergraph
 
@@ -23,14 +24,25 @@
 if [ -f $CONFIG ]; then
         . $CONFIG
 fi
- 
+
+if [ ! -d $PIDDIR ]; then
+	mkdir -p $PIDDIR
+	chown daemon:root $PIDDIR
+fi
+
 case "$1" in
   start)
 	echo -n "Starting $DESC: "
-	start-stop-daemon --start --quiet --pidfile $PIDFILE \
-		--exec $DAEMON -N 15 -c daemon:adm -- \
-		-l $MAIL_LOG -d --daemon_rrd=$RRD_DIR --rrd_name=$RRD_NAME
-	echo "$NAME."
+	# If pid file is present the daemon is probably running
+	if [ -f "$PIDFILE" ]; then
+		echo "already running."
+		exit 1
+	else
+		start-stop-daemon --start --quiet --pidfile $PIDFILE \
+			--exec $DAEMON -N 15 -c daemon:adm -- \
+			-l $MAIL_LOG -d --daemon_rrd=$RRD_DIR --rrd_name=$RRD_NAME
+		echo "$NAME."
+	fi
   ;;
 
   stop)
diff -u couriergraph-0.25/debian/postinst couriergraph-0.25/debian/postinst
--- couriergraph-0.25/debian/postinst
+++ couriergraph-0.25/debian/postinst
@@ -27,7 +27,6 @@
 #     installation fails and the `postinst' is called with `abort-upgrade',
 #     `abort-remove' or `abort-deconfigure'.
 
-RUNDIR=/var/run/servergraph
 DATADIR=/var/lib/couriergraph
 CACHEDIR=/var/cache/couriergraph
 DEFAULTS=/etc/default/couriergraph
@@ -35,9 +34,6 @@
 case "$1" in
     configure)
 
-	mkdir -p $RUNDIR || true
-	chown daemon:root $RUNDIR
-
 	if [ ! -d $DATADIR ]; then mkdir $DATADIR; fi
 	chown root:adm $DATADIR
 	chmod 775 $DATADIR
diff -u couriergraph-0.25/debian/po/pt_BR.po couriergraph-0.25/debian/po/pt_BR.po
--- couriergraph-0.25/debian/po/pt_BR.po
+++ couriergraph-0.25/debian/po/pt_BR.po
@@ -1,25 +1,19 @@
-#
-#    Translators, if you are not familiar with the PO format, gettext
-#    documentation is worth reading, especially sections dedicated to
-#    this format, e.g. by running:
-#         info -n '(gettext)PO Files'
-#         info -n '(gettext)Header Entry'
-#
-#    Some information specific to po-debconf are available at
-#            /usr/share/doc/po-debconf/README-trans
-#         or http://www.debian.org/intl/l10n/po-debconf/README-trans
-#
-#    Developers do not need to manually edit POT or PO files.
+# Debconf translations for couriergraph.
+# Copyright (C) 2012 THE couriergraph'S COPYRIGHT HOLDER
+# This file is distributed under the same license as the couriergraph package.
+# André Luís Lopes <andre...@debian.org>, 2004.
+# Adriano Rafael Gomes <adrian...@gmail.com>, 2012.
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: couriergraph\n"
+"Project-Id-Version: couriergraph 0.25-4.1\n"
 "Report-Msgid-Bugs-To: \n"
 "POT-Creation-Date: 2007-01-15 23:41+0100\n"
-"PO-Revision-Date: 2004-10-19 18:34+0200\n"
-"Last-Translator: André Luís Lopes <andre...@debian.org>\n"
-"Language-Team: Debian-BR Project <debian-l10n-portugu...@lists.debian.org>\n"
-"Language: \n"
+"PO-Revision-Date: 2012-02-26 13:39-0300\n"
+"Last-Translator: Adriano Rafael Gomes <adrian...@gmail.com>\n"
+"Language-Team: Brazilian Portuguese <debian-l10n-portuguese@lists.debian."
+"org>\n"
+"Language: pt_BR\n"
 "MIME-Version: 1.0\n"
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
@@ -28,7 +22,7 @@
 #. Description
 #: ../templates:1001
 msgid "Should CourierGraph start on boot?"
-msgstr "O CourierGraph deve ser iniciado na inicialização da máquina ?"
+msgstr "O CourierGraph deve ser iniciado na inicialização da máquina?"
 
 #. Type: boolean
 #. Description
@@ -37,9 +31,9 @@
 "Couriergraph can start on boot time as a daemon. Then it will monitor your "
 "mail logfile for changes. This is recommended."
 msgstr ""
-"O CourierGraph pode ser iniciado na inicialização da máquna como um daemon. "
-"Assim o mesmo irá monitorar seu arquivo de logs de e-mails em busca de "
-"mudanças. Este é o método recomendado."
+"O CourierGraph pode ser iniciado na inicialização da máquina como um daemon. "
+"Assim, ele monitorará seu arquivo de log de e-mails em busca de mudanças. "
+"Esse é o método recomendado."
 
 #. Type: boolean
 #. Description
@@ -53,9 +47,8 @@
 #. Type: string
 #. Description
 #: ../templates:2001
-#, fuzzy
 msgid "Logfile to be used by couriergraph:"
-msgstr "Qual arquivo de log deverá ser usado pelo couriergraph ?"
+msgstr "Arquivo de log para ser usado pelo couriergraph:"
 
 #. Type: string
 #. Description
@@ -67,20 +60,19 @@
-"Informe o arquivo de log que deverá ser usado paar criar as bases de dados "
-"para o couriergraph. Em caso de dúvidas, mantenha em branco."
+"Informe o arquivo de log que deverá ser usado para criar as bases de dados "
+"para o couriergraph. Em caso de dúvidas, mantenha o valor padrão."
 
 #. Type: boolean
 #. Description
 #: ../templates:3001
 msgid "Remove RRD files on purge?"
-msgstr "Remover arquivo RRD na remoção do pacote ?"
+msgstr "Remover arquivos RRD ao expurgar o pacote?"
 
 #. Type: boolean
 #. Description
 #: ../templates:3001
-#, fuzzy
 msgid ""
 "Couriergraph keeps its database files under /var/lib/couriergraph. State "
 "whether this directory should be removed completely on purge or not."
 msgstr ""
 "O CourierGraph mantém seus arquivos de bases de dados sob o diretório /var/"
-"lib/couriergraph. Esse diretório deverá ser completamente removido quando o "
-"pacote couriergraph for removido ?"
+"lib/couriergraph. Indique se esse diretório deverá ser completamente "
+"removido quando o pacote couriergraph for expurgado."

Reply via email to