[LTP] [PATCH v2 1/2] lib: Implement SAFE_RUN() macro (new API only)

Li Wang liwang@redhat.com
Wed Mar 25 09:27:49 CET 2020


Hi Petr,

On Mon, Mar 23, 2020 at 5:19 PM Petr Vorel <pvorel@suse.cz> wrote:

> Reviewed-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
>
Reviewed-by: Li Wang <liwang@redhat.com>


> +       switch ((rval = tst_run_cmd(argv, stdout_path, stderr_path, 1))) {
> +       case 0:
> +               break;
> +       case 255:
> +               tst_brk(TCONF, "%s:%d: %s not found in $PATH", file,
> lineno, argv[0]);
> +               break;
>

Yes, we can remove this TCONF after adding the command checking in
tst_run_cmd_fds_(). So now I think it's OK to apply this patchset and do
the remaining work separately. Thanks Petr!

-- 
Regards,
Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20200325/139db264/attachment.htm>


More information about the ltp mailing list