Re: [PATCH 2/2] gcc_full_build_factory: call contrib/test_summary after make check

[Mark Wielaard <mark@klomp.org>]

On Tue, Jan 31, 2023 at 01:56:11AM +0000, Sam James wrote:
> This gives us a nice summary of the various dejagnu results.

Looks like a nice idea in general.

But it also renames the *.sum files to .sum.sent, which means
they next bunsen step will fail to fetch them. You need to add
-t: prevents logs from being renamed

Also the "wrapper" text about emailing the sums is a little
odd imho. But it looks like there is no way to suppress that.

Cheers,

Mark

> ---
>  builder/master.cfg | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/builder/master.cfg b/builder/master.cfg
> index 71efbe9..562e794 100644
> --- a/builder/master.cfg
> +++ b/builder/master.cfg
> @@ -3259,6 +3259,13 @@ def gcc_full_build_factory_gen(extra_configure_arg=None):
>                  name='make check',
>                  timeout=3600,
>                  haltOnFailure=False, flunkOnFailure=True))
> +        # Get a useful summary (repeats the various dejagnu summaries) at
> +        # the end. From https://gcc.gnu.org/install/test.html.
> +        gcc_full_build_factory.addStep(steps.Test(
> +                workdir='gcc-build',
> +                command=['../gcc/contrib/test_summary'],
> +                name='contrib/test_summary',
> +                haltOnFailure=False, flunkOnFailure=True))
>          gcc_full_build_factory.addSteps(bunsen_logfile_upload_cpio_steps(
>                  ["*.log", "*.sum"],
>                  workdir='gcc-build'))
> -- 
> 2.39.1
>