Re: [LTP] [PATCH v3 2/2] Add centos10 to test matrix

[Petr Vorel <pvorel@suse.cz>]

Hi Siddhesh,

> gcc on CentOS 10 is configured to build with AVX on by default
> (specifically, -march=x86-64-v3), unlike other compilers, so it should
> be a useful addition to the test matrix.

> cve-2015-3290 for example fails to build on this without `-mno-avx` in
> the build flags.

I don't want to add yet another CI job just to test previous commit.
We already have quite a lot of distros (running is quite long).

@Li I suppose you want to keep centos 9 than to have centos 10.
I guess for new distros we have Fedora.

Therefore I suggest to not accept this change.

Kind regards,
Petr

> Signed-off-by: Siddhesh Poyarekar <siddhesh@gotplt.org>
> ---
>  .github/workflows/ci-docker-build.yml | 5 +++++
>  1 file changed, 5 insertions(+)

> diff --git a/.github/workflows/ci-docker-build.yml b/.github/workflows/ci-docker-build.yml
> index 44dcca055..310d8c3a3 100644
> --- a/.github/workflows/ci-docker-build.yml
> +++ b/.github/workflows/ci-docker-build.yml
> @@ -67,6 +67,11 @@ jobs:
>                CC: gcc
>                TREE: out

> +          - container: "quay.io/centos/centos:stream10"
> +            env:
> +              CC: gcc
> +              TREE: out
> +
>            - container: "debian:testing"
>              env:
>                CC: gcc

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp