[Logcheck-devel] Re: [Logcheck-commits] CVS logcheck/src

Gerfried Fuchs alfie at ist.org
Fri Jun 4 14:59:37 UTC 2004


* CVS User maks-guest <logcheck-devel at lists.alioth.debian.org> [2004-06-04 03:25]:
> no need of a .* pattern for that regex
> 
> --- /cvsroot/logcheck/logcheck/src/logcheck	2004/06/04 09:21:17	1.48
> +++ /cvsroot/logcheck/logcheck/src/logcheck	2004/06/04 09:25:58	1.49
> @@ -314,7 +314,7 @@
>  	        fi
>  
>  	        # Now apply any local-* files
> -	        for file in $(ls -1 $ignore/ | grep "local-.*") ; do
> +	        for file in $(ls -1 $ignore/ | grep "local-") ; do

 Just out of curiosity, why not make it more strict and "grep ^local-"?
Or "grep ^$ignore/local-" I guess, would be needed to try.  Not that
someone has a package called "halocal-python" or such.

 On the other hand, I'm not completely sure why we have a distinction of
the files in there anyway? Don't we apply all the files? Is that
distinction really needed? Or do I misinterpret the code in the respect?

 So long,
Alfie
-- 
Don't get mad, get interest.
                -- unknown
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.alioth.debian.org/pipermail/logcheck-devel/attachments/20040604/11bf84c0/attachment.pgp 


More information about the Logcheck-devel mailing list