Hi,

On Sun, 2007-04-15 at 16:14 -0400, Joey Hess wrote: 
> Package: devscripts
> Version: 2.10.2
> Severity: normal
> File: /usr/bin/grep-excuses
> 
> [EMAIL PROTECTED]:~>grep-excuses ikiwiki
> ikiwiki (1.48 to 1.49)
>     Maintainer: Joey Hess 
>     Too young, only 6 of 10 days old
>     Not considered
> tikiwiki (- to 1.9.7+dfsg-1)
>     Maintainer: Debian Tikiwiki team 
[...]
> I think it's a bug that it matches anything containing the package name.

The specific issue in this case is that tikiwiki's Maintainer field has
ikiwiki as a substring. I've committed the following trivial fix
requiring a whole word match in the Maintainer field, although I'm happy
to be persuaded as to whether requiring an exact match against
Maintainer would be better (and whether it should be a case-insensitive
match).

--- scripts/grep-excuses.pl     (revision 620)
+++ scripts/grep-excuses.pl     (working copy)
@@ -166,7 +166,7 @@
        $sublist=0;
        # Did the last item match?
        if ($item=~/^-?\Q$string\E\s/ or
-           $item=~/^\s*Maintainer:\s[^\n]*\Q$string\E[^\n]*$/m) {
+           $item=~/^\s*Maintainer:\s[^\n]*\b\Q$string\E\b[^\n]*$/m) {
            # In case there are embedded <li> tags
            $item =~ s%<li>%\n    %g;
            print $item;

Regards,

Adam


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

Reply via email to