<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 8, 2022 at 7:39 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If test exits on time (i.e. no timeout) kill in _tst_cleanup_timer()<br>
have nothing to kill therefore following wait exits 143.<br>
<br>
set -e (or #!/bin/sh -e or set -o errexit) quits on any non-zero exit code,<br>
fix hardens _tst_cleanup_timer() to be able to be used on scripts with it.<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
I can use 'if ...; then ; fi' if you prefer:<br>
<br>
        if [ "$LTP_COLORIZE_OUTPUT" = "n" -o "$LTP_COLORIZE_OUTPUT" = "0" ]; then<br>
                return 0<br>
        fi<br>
<br>
        if [ "$LTP_COLORIZE_OUTPUT" = "y" ] || [ "$LTP_COLORIZE_OUTPUT" = "1" ]; then<br>
                return 1<br>
        fi<br>
<br>
...<br>
        if [ "$color" = 1 ]; then<br>
                tst_flag2color "$1"<br>
        fi<br>
        printf "$2"<br>
        if [ "$color" = 1 ]; then<br>
                printf '\033[0m'<br>
        fi<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">These ^ clarifications should be added in patch 2/4, right?</div><br></div><div><div class="gmail_default" style="font-size:small">Anyway, this one looks good.</div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>></div></div></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>