[dpdk-dev] [PATCH v2 3/3] ci: add travis ci support for native ppc64le
dwilder
dwilder at us.ibm.com
Wed Feb 19 21:32:41 CET 2020
On 2020-02-19 05:46, David Marchand wrote:
> On Wed, Feb 19, 2020 at 2:10 AM David Wilder <dwilder at 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 at us.ibm.com>
>> Reviewed-by: David Christensen <drc at 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.
More information about the dev
mailing list