Re: [dpdk-dev] [PATCH v2 3/3] ci: add travis ci support for native ppc64le

[David Marchand <david.marchand@redhat.com>]

On Wed, Feb 19, 2020 at 2:10 AM David Wilder <dwilder@us.ibm.com> wrote:
>
> This change follows the example of aarch64 Travis support
> by adding support for ppc64le gcc builds.  Limitations for
> ppc64le are the same as aarch64 as described in commit
> 31bb45bcfd.

checkpatch did not catch it because of the \n (?).
When referring to other commits, please use the format 'commit <12+
chars of sha1> ("<title line>")'.


>
> Signed-off-by: David Wilder <dwilder@us.ibm.com>
> Reviewed-by: David Christensen <drc@linux.vnet.ibm.com>

I can see no clang compilation.
Is it unsupported? forgotten? unwanted?


> ---
>  .travis.yml | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
>
> diff --git a/.travis.yml b/.travis.yml
> index 1d43f16ae..c15bf4f07 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -153,5 +153,35 @@ matrix:
>          packages:
>            - *extra_packages
>            - *libabigail_build_packages
> +  - env: DEF_LIB="static"
> +    arch: ppc64le
> +    compiler: gcc
> +    addons:
> +      apt:
> +        packages:
> +          - *required_packages

Copy/pasting this part is useless.
The addons: global node would make all jobs use this required_packages
as a default.
I have a patch cleaning this for arm.


> +  - env: DEF_LIB="shared"
> +    arch: ppc64le
> +    compiler: gcc
> +    addons:
> +      apt:
> +        packages:
> +          - *required_packages
> +  - env: DEF_LIB="shared" OPTS="-Denable_kmods=false" BUILD_DOCS=1
> +    arch: ppc64le
> +    compiler: gcc
> +    addons:
> +      apt:
> +        packages:
> +          - *required_packages
> +          - *doc_packages
> +  - env: DEF_LIB="shared" EXTRA_PACKAGES=1 ABI_CHECKS=1
> +    arch: ppc64le
> +    compiler: gcc
> +    addons:
> +      apt:
> +        packages:
> +          - *extra_packages
> +          - *libabigail_build_packages
>
>  script: ./.ci/${TRAVIS_OS_NAME}-build.sh
> --
> 2.25.0
>


-- 
David Marchand