[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: grep-execuses -- testing update excuses utility



Today, Joey Hess <joeyh@debian.org> wrote:
> This is trivial, but maybe it will save someone else the bother. I'm
> running it from a nightly cron job.

Nice script, but I think that:

		if ($item=~/$string/) {
should better be written as (conditionally, via a command-line option?):
                if ($item=~/*\s$string\s/) {

lest "grep-excuses ed" (or any package name that is a substring of any
other package name) gets messy...

regards,
-- 
Andreas Fuchs, <asf@acm.org>, <d96001@htlwrn.ac.at>, antifuchs
Hail RMS! Hail Cthulhu! Hail Eris! All hail Discordia!

Attachment: pgpKMJUYo336G.pgp
Description: PGP signature


Reply to: