<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 21, 2018 at 5:12 PM, Richard Palethorpe <span dir="ltr"><<a href="mailto:rpalethorpe@suse.de" target="_blank">rpalethorpe@suse.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<div><div class="h5"><br>
Cyril Hrubis writes:<br>
<br>
> Recently we had a problem when openQA (test automation framework) passed<br>
> wrong device path as LTP_BIG_DEV. It has been much more complicated to<br>
> debug because the error message from mkfs was not shown at all, since<br>
> the tst_mkfs uses ROD_SILENT to execute the mkfs command.<br>
><br>
> The best solution for ROD_SILENT is to store the command output into a<br>
> variable and print it only on failure which is done by this patch.<br>
><br>
> Signed-off-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>><br>
> ---<br>
>  testcases/lib/test.sh     | 3 ++-<br>
>  testcases/lib/tst_test.sh | 3 ++-<br>
>  2 files changed, 4 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/testcases/lib/test.sh b/testcases/lib/test.sh<br>
> index 341c1f0f4..bce9893a9 100644<br>
> --- a/testcases/lib/test.sh<br>
> +++ b/testcases/lib/test.sh<br>
> @@ -225,8 +225,9 @@ tst_timeout()<br>
><br>
>  ROD_SILENT()<br>
>  {<br>
> -     $@ > /dev/null 2>&1<br>
> +     local tst_out=$($@ 2>&1)<br>
>       if [ $? -ne 0 ]; then<br>
> +             echo "$tst_out"<br>
>               tst_brkm TBROK "$@ failed"<br>
>       fi<br>
>  }<br>
> diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh<br>
> index d6b638549..48afb9cc4 100644<br>
> --- a/testcases/lib/tst_test.sh<br>
> +++ b/testcases/lib/tst_test.sh<br>
> @@ -118,8 +118,9 @@ tst_brk()<br>
><br>
>  ROD_SILENT()<br>
>  {<br>
> -     tst_rod $@ > /dev/null 2>&1<br>
> +     local tst_out=$(tst_rod $@ 2>&1)<br>
>       if [ $? -ne 0 ]; then<br>
> +             echo "$tst_out"<br>
>               tst_brk TBROK "$@ failed"<br>
>       fi<br>
>  }<br>
> --<br>
> 2.13.6<br>
<br>
</div></div>LGTM except that you could maybe use ${ } instead of $( ) as a small<br>
optimisation.<br></blockquote><div><span style="font-family:arial,helvetica,sans-serif"><font size="2"><br></font></span><div class="gmail_default" style="display:inline"><span style="font-family:arial,helvetica,sans-serif"><font size="2">​For ​</font></span></div><span style="font-family:arial,helvetica,sans-serif"><font size="2">what optimisation?<br><br>$(command) runs the command and captures its output in separate shell.<br>${parameter} the value of parameter is substituted.<br><br><br></font></span><div class="gmail_default" style="display:inline"><span style="font-family:arial,helvetica,sans-serif"><font size="2">​Here the purpose is to hide the output in sub shell I guess, shouldn't use $()?</font></span></div></div></div><br><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Li Wang<br><a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a></div>
</div></div>