Re: [PATCH] Fix builder step name.

[Mark Wielaard <mark@klomp.org>]

Hi Martin,

On Thu, Jun 16, 2022 at 07:08:49AM +0200, Martin Liška wrote:
> Change builder step like:
> make -k checkCC_FOR_TARGET=clang CXX_FOR_TARGET=cl
> 
> to
> make -k check CC_FOR_TARGET=clang CXX_FOR_TARGET=cl

Looks good to me. Please push.

Thanks,

Mark

> ---
>  builder/master.cfg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/builder/master.cfg b/builder/master.cfg
> index 8455dfe..e49b2fa 100644
> --- a/builder/master.cfg
> +++ b/builder/master.cfg
> @@ -1638,7 +1638,7 @@ def binutils_gdb_factory_factory(runtestflags=None):
>                  make_check_command.append('RUNTESTFLAGS='+runtestflags)
>          binutils_gdb_factory.addStep(steps.Test(
>                  command=make_check_command,
> -                name=('make -k check' + (runtestflags if runtestflags else ''))[:50], # length limit
> +                name=('make -k check' + (' ' + runtestflags if runtestflags else ''))[:50], # length limit
>                  timeout=14400, # 4 hours should be enough to get some output
>                  haltOnFailure=False, flunkOnFailure=False))  # test failure = ok
>          binutils_gdb_factory.addStep(steps.ShellCommand(
> -- 
> 2.36.1
>