From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Larmour To: overseers@sources.redhat.com Subject: Re: [Fwd: failure notice] Date: Wed, 30 Aug 2000 14:13:00 -0000 Message-ID: <39AD7904.15829906@redhat.com> References: <39AD4E83.55F4D362@redhat.com> <20000830113053.A23654@shell17.ba.best.com> <20000830120321.A3776@shell17.ba.best.com> <20000830153622.B27245@cygnus.com> <20000830131422.A26689@shell17.ba.best.com> <39AD73E6.5A848A10@redhat.com> <20000830170040.A28076@cygnus.com> X-SW-Source: 2000-q3/msg00329.html Message-ID: <20000830141300.bTXAorqV37htfGjw0t231PRyi5YG7WAAu31Sa0923PA@z> 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