Re: git: 6024564cd4da - main - Cirrus-CI: split main script into separate world + kernel

From: Alexander Richardson <arichardson_at_freebsd.org>
Date: Fri, 09 Jun 2023 15:26:40 UTC
On Fri, 9 Jun 2023, 08:08 Ed Maste, <emaste@freebsd.org> wrote:

> The branch main has been updated by emaste:
>
> URL:
> https://cgit.FreeBSD.org/src/commit/?id=6024564cd4da1f7a24c7e13a4aa6b04707eafb60
>
> commit 6024564cd4da1f7a24c7e13a4aa6b04707eafb60
> Author:     Ed Maste <emaste@FreeBSD.org>
> AuthorDate: 2023-06-09 13:53:08 +0000
> Commit:     Ed Maste <emaste@FreeBSD.org>
> CommitDate: 2023-06-09 15:07:24 +0000
>
>     Cirrus-CI: split main script into separate world + kernel
>
>     It appears that Cirrus-CI has a 100MB limit for log output, and we
>     exceed that (!) with the amd64-gcc12 build.  Separate world and kernel
>     build tasks in an attempt to stay below the limit.
>
>     This also has the benefit of showing world and kernel build status
>     separately in the Cirrus-CI UI.
>
>     PR:             271903
>     Sponsored by:   The FreeBSD Foundation
> ---
>  .cirrus.yml | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index 12d78f465c55..d22b5f189f53 100644
> --- a/.cirrus.yml
> +++ b/.cirrus.yml
> @@ -73,8 +73,11 @@ task:
>    - mkdir -p /usr/obj/$(pwd -P)
>    - chown user:user /usr/obj/$(pwd -P)
>
> -  script:
> -  - su user -c "make -j$(sysctl -n hw.ncpu) CROSS_TOOLCHAIN=${TOOLCHAIN}
> WITHOUT_TOOLCHAIN=yes buildworld buildkernel"
> +  build_world_script:
> +  - su user -c "make -j$(sysctl -n hw.ncpu) CROSS_TOOLCHAIN=${TOOLCHAIN}
> WITHOUT_TOOLCHAIN=yes buildworld"
> +
> +  build_kernel_script:
> +  - su user -c "make -j$(sysctl -n hw.ncpu) CROSS_TOOLCHAIN=${TOOLCHAIN}
> WITHOUT_TOOLCHAIN=yes buildkernel"
>

Maybe these commands should just be using make -s? I believe make should
now print the failed command so there should be no need to print all of
them.

Alex