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

Bug#241236: [patch] ignore files in ignore.d.violations without prefix



Package: logcheck
Version: 1.2.16
Severity: wishlist
Tags: patch


hello steve kemp,

this patches allows the user to _really_ write his own
files in ignore.d.violations
previous they had to have the prefix logcheck-,
which isn't handy, not documented and therefor seems to be a bug .. 
took some time to figure out. ;)

as a side note logcheck install rules in this dir without logchek-
prefix like su.

as a bonus attached patch catches also a typing fix.
regards maks


ps tested on my machine, works like charm.

--- logcheck-1.2.16/src/logcheck	2004-03-03 20:39:53.000000000 +0100
+++ logcheck-1.2.16-maks/src/logcheck	2004-03-31 15:29:48.000000000 +0200
@@ -233,7 +233,7 @@
 
     RETURN=1
 
-    for grepfile in $(ls -1 $raise | grep -v "logcheck-.*"); do
+    for grepfile in $(ls -1 $raise); do
 	debug "greplogoutput: $grepfile"
 
 	egrep --text -f $raise/$grepfile $TMPDIR/logoutput-sorted \
@@ -252,8 +252,8 @@
 	    if [ "$(basename $grepfile)" = "logcheck" ]; then 
 
 		debug "Applying Logcheck override files"
-		# Now ignore all entries from the locgheck-<package> files
-		for file in $(ls -1 $ignore/ | grep "logcheck-.*") ; do
+		# Now ignore all entries from the logcheck-<package> files
+		for file in $(ls -1 $ignore/) ; do
 		    debug "clean logcheck-<package>: $file"
 		    egrep --text -v -f $ignore/$file $TMPDIR/checked \
 			>> $TMPDIR/checked.1

Attachment: signature.asc
Description: Digital signature


Reply to: