On Wed, 6 Jan 2010 at 11:48, Christoph Hellwig wrote:
> > 2) convert "let..." into something (hopefully) more portable
> Dave converted these away from expr due to performance reasons. I'd
> like too see a prove that performance hasn't regressed due to this
That's why I hesitated to just change all scripts to /bin/bash, but didn't
dare to say so, because of the inevitable "but today's computers are fast
enough" reactions :-)
> > 3) replace 'a == b' with 'a = b' in bourne shell scripts
> This looks fine. If you want feel free to submit these as a separate
> first patch so that we have the large pile sorted out.
OK, will do.
> I fear this might cause some overhead in the shell. What about the
> following instead:
> for i in `seq 0 39`; do
> ln -s $o symlink_$i
I didn't go for seq(1), as I feared this wouldn't be available on every
machine. Given that the xfstests scripts have a lot of "what if we're
running on IRIX" conditions, I just wasn't sure if seq(1) would be
available there. But yes, I'd much rather like to do this with seq(1) as
well, if this is OK to everybody.
> > --- xfstests.orig/032 2010-01-03 00:42:16.601617592 -0800
> > +++ xfstests/032 2010-01-03 00:43:56.321617592 -0800
> > @@ -66,6 +66,7 @@ do
> > [ $fs = ext3 ] && preargs="-F"
> > [ $fs = ext4 ] && preargs="-F"
> > [ $fs = ext4dev ] && preargs="-F"
> > + [ $fs = nilfs2 ] && preargs="-q"
> This hunk is unrelated. Please submit it with a trivial one-liner
> description and a signed-off-by line and I'll put it in.
OK, will do.
> > -echo -e -n "\n\r*** XFS QA 044 - done\n\r\n\r" >/dev/console
> > +printf "\n\r*** XFS QA 044 - done\n\r\n\r" >/dev/console
> This looks okay, but should be mentioned in the patch description.
Will do. It's needed, because sh(1)'s echo builtin doesn't know about -e.
> > done
> > -popd >/dev/null
> Same here.
Will do. Standard sh(1) doesn't know popd/pushd either.
BOFH excuse #398:
Data for intranet got routed through the extranet and landed on the internet.