On Fri, Sep 01, 2023 at 10:48:10AM +0200, Marek Vasut wrote:

> Test both 32bit and 64bit sandbox boards in CI.
> 
> Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
> ---
> Cc: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Tom Rini <tr...@konsulko.com>
> ---
> DEPENDS:
> - [v2] test: print: Fix hexdump test on 64bit systems
> - test: dm: test-fdt: Use fdtdec_get_int() in dm_check_devices()
> - configs: sandbox64: Enable legacy image format support

Nice work.

Reviewed-by: Tom Rini <tr...@konsulko.com>

> ---
>  .gitlab-ci.yml | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 6d7ffdd1bbc..a7282be3181 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -291,6 +291,23 @@ sandbox without LTO test.py:
>      BUILD_ENV: "NO_LTO=1"
>    <<: *buildman_and_testpy_dfn
>  
> +sandbox64 test.py:
> +  variables:
> +    TEST_PY_BD: "sandbox64"
> +  <<: *buildman_and_testpy_dfn
> +
> +sandbox64 with clang test.py:
> +  variables:
> +    TEST_PY_BD: "sandbox64"
> +    OVERRIDE: "-O clang-16"
> +  <<: *buildman_and_testpy_dfn
> +
> +sandbox64 without LTO test.py:
> +  variables:
> +    TEST_PY_BD: "sandbox64"
> +    BUILD_ENV: "NO_LTO=1"
> +  <<: *buildman_and_testpy_dfn

I think this means it'll be time to split sandbox out in to its own
stage on Azure as we can get 10 meaningful-but-long runs there and so
it's best for overall job time to make that a stage. I'll go give that a
spin shortly unless you want to.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to