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

[dwilder <dwilder@us.ibm.com>]

On 2020-02-19 05:46, David Marchand wrote:
> 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>")'.
> 

Ok

> 
>> 
>> 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?
> 

Unsupported for now (issues with altivec).

> 
>> ---
>>  .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.

I see your cleanup patch now.  I will refactor the ppc support on top 
and let you know if I have issues.

Thanks for the review.