Bug#494682: [Pkg-citadel-devel] Bug#494682: Working on this bug

2008-08-12 Thread Michael Meskes
On Mon, Aug 11, 2008 at 07:12:47PM -0300, Thadeu Lima de Souza Cascardo wrote:
 Please, wait until tomorrow, Aug 12th, by 17:00 UTC, which is when I'm
 going to ask for the sponsorship and attach the NMU diff. If I don't do
 that by that time, you may start working on it.

Could it be you got the order of things slightly reversed? You ask the
maintainers to not upload before you can ask someone to upload your NMU? Don't
you think the right way is to send the patch and give us a chance to upload
ourselves? 

I do value your effort of creating the patch which reduces our workload but it
appears to me I misunderstood your email.

Michael
-- 
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: [EMAIL PROTECTED]
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#494682: [Pkg-citadel-devel] Bug#494682: Working on this bug

2008-08-12 Thread Thadeu Lima de Souza Cascardo
On Tue, Aug 12, 2008 at 03:29:02PM +0200, Michael Meskes wrote:
 On Mon, Aug 11, 2008 at 07:12:47PM -0300, Thadeu Lima de Souza Cascardo wrote:
  Please, wait until tomorrow, Aug 12th, by 17:00 UTC, which is when I'm
  going to ask for the sponsorship and attach the NMU diff. If I don't do
  that by that time, you may start working on it.
 
 Could it be you got the order of things slightly reversed? You ask the
 maintainers to not upload before you can ask someone to upload your NMU? Don't
 you think the right way is to send the patch and give us a chance to upload
 ourselves? 
 
 I do value your effort of creating the patch which reduces our workload but it
 appears to me I misunderstood your email.
 
 Michael
 -- 
 Michael Meskes
 Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
 ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: [EMAIL PROTECTED]
 Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!

Yes, I thought of attaching the patch, which I am doing right now. I am
sorry for not being nice and doing it. I just wanted to upload the
Brazillian Portuguese translation too and avoid having to do two uploads
in the case you didn't want to wait for it. Anyway, this is no excuse
for not sending the patch. Since you seem to be pretty active, I am just
sending the patch and let you revise and upload it with your
modifications, instead of asking for someone else's sponshorship.

Thank you very much for your work and reminding me that we all should be
nicer with our fellow Debian Developers/Maintainers.  :-)

This patch does not include the pt_BR translation because I didn't get
it from my friend in the timeframe I expected.

Thadeu Cascardo.
diff -u citadel-7.37/debian/citadel-common.postinst 
citadel-7.37/debian/citadel-common.postinst
--- citadel-7.37/debian/citadel-common.postinst
+++ citadel-7.37/debian/citadel-common.postinst
@@ -9,7 +9,7 @@
 case $1 in
 configure)
 if ! getent group citadel /dev/null; then 
-groupadd citadel
+addgroup --system citadel
 fi
 
 if ! getent passwd citadel /dev/null; then 
diff -u citadel-7.37/debian/citadel-server.templates 
citadel-7.37/debian/citadel-server.templates
--- citadel-7.37/debian/citadel-server.templates
+++ citadel-7.37/debian/citadel-server.templates
@@ -14,7 +14,7 @@
 _Description: Enable external authentication mode?
  Please choose the user authentication mode. By default Citadel will
  use its own internal user accounts database. If you accept this
- option, Citadel users will have accounts on the host system, 
+ option, Citadel users will have accounts on the host system,
  authenticated via /etc/passwd (or LDAP).
  .
  Do not accept this option unless you are sure it is required, since
diff -u citadel-7.37/debian/changelog citadel-7.37/debian/changelog
--- citadel-7.37/debian/changelog
+++ citadel-7.37/debian/changelog
@@ -1,3 +1,13 @@
+citadel (7.37-1.1) unstable; urgency=low
+
+  * NMU.
+  * Adds a system group instead of a regular one (Closes: #494682)
+  * Fix a typo in the debconf po template and update the finnish
+translation (Closes: #492611)
+  * Updated the russian translation (Closes: #492386)
+
+ -- Thadeu Lima de Souza Cascardo [EMAIL PROTECTED]  Mon, 11 Aug 2008 
17:59:37 -0300
+
 citadel (7.37-1) unstable; urgency=low
 
   [ Wilfried Goesgens ]
diff -u citadel-7.37/debian/po/de.po citadel-7.37/debian/po/de.po
--- citadel-7.37/debian/po/de.po
+++ citadel-7.37/debian/po/de.po
@@ -53,7 +53,7 @@
 msgid 
 Please choose the user authentication mode. By default Citadel will use its 
 own internal user accounts database. If you accept this option, Citadel 
-users will have accounts on the host system,  authenticated via /etc/passwd 
+users will have accounts on the host system, authenticated via /etc/passwd 
 (or LDAP).
 msgstr 
 Bitte w�hlen Sie die Authentifizierungsmethode f�r Benutzer. Standardm��ig 
diff -u citadel-7.37/debian/po/templates.pot 
citadel-7.37/debian/po/templates.pot
--- citadel-7.37/debian/po/templates.pot
+++ citadel-7.37/debian/po/templates.pot
@@ -50,7 +50,7 @@
 msgid 
 Please choose the user authentication mode. By default Citadel will use its 
 own internal user accounts database. If you accept this option, Citadel 
-users will have accounts on the host system,  authenticated via /etc/passwd 
+users will have accounts on the host system, authenticated via /etc/passwd 
 (or LDAP).
 msgstr 
 
diff -u citadel-7.37/debian/po/vi.po citadel-7.37/debian/po/vi.po
--- citadel-7.37/debian/po/vi.po
+++ citadel-7.37/debian/po/vi.po
@@ -52,7 +52,7 @@
 msgid 
 Please choose the user authentication mode. By default Citadel will use its 
 own internal user accounts database. If you accept this option, Citadel 
-users will have accounts on the host system,  authenticated via /etc/passwd 
+users will have accounts on the host system, authenticated via /etc/passwd 
 (or LDAP).
 msgstr 
 
diff -u citadel-7.37/debian/po/ru.po