Why are you using -E in the grep call here? $DEV should not contain any
   regular expression, it's just a device name.

An oversight on my part.

I was originally trying to make sure I don't match together "/dev/foo" and "/dev/foobar" using regex but I got conflicted output between bash and dash so just used the space and forgot to remove the -E :)

Same patch below with -E removed. Don't mind if it doesn't get used, it just isn't often I get a chance to "give back".

--- standard_3.0pl1-121 2012-03-21 08:34:12.339516047 +0000
+++ standard    2012-03-21 08:34:02.111770395 +0000
@@ -10,6 +10,7 @@
 cd /
 LOCKFILE=/var/lock/cron.daily
 LOFO=lost+found
+SEENFS=""

 # When flock is available, avoid running more than once at a time
 if `which flock >/dev/null`; then
@@ -41,6 +42,10 @@
     # Only check on FS where we might expect lost+found
     echo "$FSTYPE" | grep -q -E '^(ext2|ext3|ext4|xfs)$' || continue

+    # Only check a filesystem once
+    echo "$SEENFS" | grep -q "$DEV " && continue
+    SEENFS="$DEV $SEENFS"
+
     [ "$MTPT" = '/' ] && MTPT=""
     # Replace spaces in the path (\040)
     MTPT="`echo $MTPT | sed -e 's/\\040/ /g'`"



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to