Bug#368098: Intend to NMU debroster for this bug

2006-06-01 Thread Thijs Kinkhorst
reopen 368098
thanks

On Wed, 2006-05-31 at 22:17 +0100, Matthew Vernon wrote:
 severity 368098 important
 quit
 
  If you do not approve of this, please let me know.
 
 I don't. Not least of which, your patch doesn't actually solve the
 problem of not properly uniqifying the maintainer list.

Your patch does not work for the case of Brian Nelson [EMAIL PROTECTED]
and Brian Nelson [EMAIL PROTECTED], who are two completely different
developers. I therefore still think that the only right way is to
uniqify by the full string.


Thijs


signature.asc
Description: This is a digitally signed message part


Bug#368098: Intend to NMU debroster for this bug

2006-05-31 Thread Thijs Kinkhorst
Hello Matthew,

This package hasn't seen an update in 1.5 years and now has this RC bug.
I intend to NMU debroster anytime soon for this RC-bug with the attached
patch. I've completely removed the key specification for sort because it
eliminates valid contributors. The only possible risk is that a
maintainer which has different email addresses is mentioned twice but
this is not avoidable; see e.g. Brian Nelson and Brian Nelson.

Since the package hasn't been updated for a long time, I also plan to
fix open bugs in the BTS and any lintian problems I encounter. I will
submit a patch for that when it's ready.

If you do not approve of this, please let me know.


Thijs
diff -ur debroster-1.12.orig/dr.sh debroster-1.12/dr.sh
--- debroster-1.12.orig/dr.sh	2003-11-11 16:45:38.0 +0100
+++ debroster-1.12/dr.sh	2006-05-31 12:31:16.0 +0200
@@ -22,6 +22,6 @@
 scroll [EMAIL PROTECTED]
 }
 
-/var/lib/dpkg/available sed -ne 's/^Maintainer: //p' | sort -u -k1.1,2.0 | build_arg_list
+/var/lib/dpkg/available sed -ne 's/^Maintainer: //p' | sort -u | build_arg_list
 
 exit 0


signature.asc
Description: This is a digitally signed message part


Bug#368098: Intend to NMU debroster for this bug

2006-05-31 Thread Matthew Vernon
severity 368098 important
quit

 If you do not approve of this, please let me know.

I don't. Not least of which, your patch doesn't actually solve the
problem of not properly uniqifying the maintainer list.

Matthew

-- 
At least you know where you are with Microsoft.
True. I just wish I'd brought a paddle.
http://www.debian.org


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