[darcs-users] darcs patch: accept issue1620: record lies about leav... (and 1 more)

Trent W. Buck trentbuck at gmail.com
Tue Sep 29 14:46:05 UTC 2009


Jason, the test scripts look OK to apply from visual inspection.
Nitpicks follow.

> LOGFILE=`grep "Logfile left in" out | sed "s/Logfile left in //" | sed
> s/.$//`

I would use a single awk or sed command here, e.g.

    sed -n '/Logfile left in/ { s/Logfile left in //; s/\.$//; p}' out

> test -e "$LOGFILE"

Testing the $LOGFILE is a file as well (test -f) wouldn't hurt.

> grep 'new log' $LOGFILE

Likewise, grep '^new log$' or fgrep -x 'new log' would be more specific.
I'm thinking of a hypothetical case where the logfile manages to contain
an error like "bash: echo 'new log' >foo: command not found".



More information about the darcs-users mailing list