Re: [Fwd: failure notice]

[Jonathan Larmour <jlarmour@redhat.com>]

Chris Faylor wrote:
> 
> On Wed, Aug 30, 2000 at 09:51:50PM +0100, Jonathan Larmour wrote:
> >Looking at check-for-listname.sh (thanks for the pointer Crash), the
> >relevant line is:
> >if cat $TMPFILE | sed -n '/^[Cc][Cc]: /,/^[a-zA-Z]/p' | sed '$d' |
> >   egrep -i "$VALID_LISTNAME" >/dev/null 2>&1
> >
> >If the "sed '$d' |" is removed, it works correctly on this mail. I can't
> >see any reason for the "sed '$d'" to be there. Is there one? If not, I'll
> >just remove it.
> 
> I think that the $d guards against something like:
> 
> To: greatopportunities@foryou.com,
>     dontmiss@this.com
> Subject: ecos-discuss@sourceware.cygnus.com
> 
> Admittedly, it's not likely that something like this would happen, but this
> should be moot anyway. 

That's not what it's for either - sed only displays the line beginning with
the CC. I even just tried it to be sure.

Anyway, no matter - whatever works :-).

>  I've checked in a patch which should solve this problem. I've checked
> it on the header in question and it seems to work fine.

As long as formail isn't slow. Probably not.
 
> Now, let's see if there are thousands of angry screams when we find that this
> only works for ecos-discuss@sourceware.cygnus.com...

Heh.

Jifl
-- 
Red Hat, 35 Cambridge Place, Cambridge, UK. CB2 1NS  Tel: +44 (1223) 728762
"Plan to be spontaneous tomorrow."  ||  These opinions are all my own fault