mbox series

[v2,0/2] add travis ci support for aarch64

Message ID 20191220093709.83992-1-ruifeng.wang@arm.com (mailing list archive)
Headers
Series add travis ci support for aarch64 |

Message

Ruifeng Wang Dec. 20, 2019, 9:37 a.m. UTC 
  This patch set is to enable native aarch64 build in Travis CI.
It leverages Travis CI multi arch support.

As the first step, compilation jobs are added.
Unit test is not added for now due to service limitation. We are
planning to run unit test with no-huge in future.

This patch set has dependency on:
http://patches.dpdk.org/patch/63969/


v2:
Removed dist designation from matrix since base dist is changing.
Add explanation for library path issue.

Ruifeng Wang (2):
  ci: add travis ci support for aarch64
  devtools: add path to additional shared object files

 .ci/linux-setup.sh    | 11 +++++++----
 .travis.yml           | 37 ++++++++++++++++++++++++++++++++++++-
 devtools/test-null.sh |  2 +-
 3 files changed, 44 insertions(+), 6 deletions(-)

Comments

David Marchand Dec. 20, 2019, 1:57 p.m. UTC | #1 
On Fri, Dec 20, 2019 at 10:38 AM Ruifeng Wang <ruifeng.wang@arm.com> wrote:
>
> This patch set is to enable native aarch64 build in Travis CI.
> It leverages Travis CI multi arch support.
>
> As the first step, compilation jobs are added.
> Unit test is not added for now due to service limitation. We are
> planning to run unit test with no-huge in future.
>
> This patch set has dependency on:
> http://patches.dpdk.org/patch/63969/
>
>
> v2:
> Removed dist designation from matrix since base dist is changing.

*If* (I did not think about the implications yet, hence the *If*) we
switch the base distribution to Bionic, we can't push it anyway
without your second patch.

Patches order must be reversed.
And for the jobs in travis, the distribution for aarch64 native jobs
must be explicitely bionic distribution (you did not update the
commitlog in the v2 but not a problem if you restore the dist: bionic
entries).
We can then take your series.

Kevin patch that upgrades globally from xenial to bionic, can then be
integrated after yours.


> Add explanation for library path issue.

Thanks for this.