On Mon, Oct 06, 2003 at 01:10:38AM +0200, Frank Lichtenheld wrote:
> In the Special Maintainers list of cron/people_scripts/people.pl are
> again some maintainer team names missing:
> Debian GNOME Maintainers
> Debian LDP Maintainers
> Debian KDE/Qt Maintainers
> 
> Attached is a patch that handles these names in a more generic way
> (untested).

I have two problems with your patch:

 * it includes the team packages in the second list, which is for
   individuals, and

 * it 

> Index: people.pl
> ===================================================================
> RCS file: /cvs/webwml/cron/people_scripts/people.pl,v
> retrieving revision 1.12
> diff -u -r1.12 people.pl
> --- people.pl 16 Sep 2003 12:16:35 -0000      1.12
> +++ people.pl 5 Oct 2003 23:03:08 -0000
> @@ -14,11 +14,8 @@
>       "Debian QA Group",
>       "Debian Install System Team",
>       "Debian Policy List",
> -     "Debian GCC maintainers",
> -     "Debian GNUstep maintainers",
>       "GNU Hurd Maintainers",
>       "GNU Libc Maintainers",
> -     "Debian Apache Maintainers",
>       "Debbugs developers",
>       "Dpkg Development",
>       "teTeX maintainers",
> @@ -27,9 +24,6 @@
>       "Dynamic DNS Tools and Services",
>       "Debian OpenOffice Team",
>       "Cdbs Hackers",
> -     "Debian Java Maintainers",
> -     "Debian OCaml Maintainers",
> -     "Debian SDL maintainers",
>  );
>  
>  
> @@ -218,7 +212,10 @@
>               }
>               $maintainer = from_utf8_or_iso88591_to_sgml($maintainer);
>  # Take care of the annoying cases and exceptions and overrides and stuff
> -             if ($maintainer =~ /Debian Quality Assurance.*<(.+)>/) {
> +             if ($maintainer =~ /(Debian \S+ [Mm]aintainers).*<(.+)>/) {
> +                     $lastname = $1; $firstname = ''; $email = $2;
> +             }
> +             elsif ($maintainer =~ /Debian Quality Assurance.*<(.+)>/) {
>                       $lastname = 'Debian QA Group'; $firstname = ''; $email 
> = $1;
>               }
>               elsif ($maintainer =~ /Boot Floppies Team <(.+)>/) {


-- 
Matt

Reply via email to