[patch] testsuite/testing.sh: removes obvious bashisms

Cristian Ionescu-Idbohrn cristian.ionescu-idbohrn at axis.com
Fri May 2 03:13:12 PDT 2008


On Fri, 2 May 2008, Denys Vlasenko wrote:

> Scary mess.

:)

> Either declare than we absolutely require echo -ne to work.

Easy.  Force the shebang lines to (in all scripts):

  #!/bin/bash

and leave those who don't have bash installed do whatever they can.

> Making it work without echo -ne will hurt readability too much.

Realy?  Why?  Readability pain from using:

  # printf "foo\tbar"

compared to:

  # echo -ne "foo\tbar"

is minimal (if any), IMHO, make the scripts more maintainable accross
various distributions and make more people happy :)

> Or have "echo-n", "echo-ne" and "echo-e" helper programs written in C. 8]

I have no problems with that, but I still think printf is easier and more
flexible.


Cheers,

-- 
Cristian


More information about the busybox mailing list