Thread (1 message) 1 message, 1 author, 2012-11-02

Re: [PATCH] fdtget-runtest.sh: Fix failures when /bin/sh isn't bash

From: Mike Frysinger <hidden>
Date: 2012-11-02 20:26:44

On Tuesday 20 March 2012 22:23:46 Stephen Warren wrote:
quoted hunk
On Ubuntu, /bin/sh is dash (at least by default), and dash's echo doesn't
accept the -e option. This means that fdtget-runtest.sh's EXPECT file will
contain "-e foo" rather than just "foo", which causes a test failure.

To work around this, run /bin/echo instead of (builtin) echo, which has
more chance of supporting the -e option.

Another possible fix is to change all the #! lines to /bin/bash rather
than /bin/sh, and change run_tests.sh to invoke sub-scripts using $SHELL
instead of just "sh". However, that would require bash specifically, which
may not be desirable.
--- a/tests/fdtget-runtest.sh
+++ b/tests/fdtget-runtest.sh

-echo -e $expect >$EXPECT
+/bin/echo -e $expect >$EXPECT
the better fix is to use printf and %b:
	printf '%b\n' "$expect" > $EXPECT
-mike
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help