ci: fix telemetry dependency in Travis
Checks
Commit Message
From: "juraj.linkes" <juraj.linkes@pantheon.tech>
libjansson4 is not enough to build telemetry. Replace it with
libjansson-dev.
Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
juraj.linkes@pantheon.tech writes:
> From: "juraj.linkes" <juraj.linkes@pantheon.tech>
>
> libjansson4 is not enough to build telemetry. Replace it with
> libjansson-dev.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
I see that earlier builds would print:
=================
Content Skipped
=================
libs:
telemetry: missing dependency "jansson"
But with this patch, I see that telemetry is included in the built libs.
Thanks for the catch.
Acked-by: Aaron Conole <aconole@redhat.com>
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of
> juraj.linkes@pantheon.tech
> Sent: Friday, April 3, 2020 8:46 PM
> To: aconole@redhat.com; maicolgabriel@hotmail.com
> Cc: dev@dpdk.org; juraj.linkes <juraj.linkes@pantheon.tech>
> Subject: [dpdk-dev] [PATCH] ci: fix telemetry dependency in Travis
>
> From: "juraj.linkes" <juraj.linkes@pantheon.tech>
>
> libjansson4 is not enough to build telemetry. Replace it with libjansson-dev.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
> .travis.yml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index fd4f79cfc..2d2292ff6 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -15,7 +15,7 @@ addons:
> update: true
> packages: &required_packages
> - [libnuma-dev, python3-setuptools, python3-wheel, python3-pip, ninja-
> build]
> - - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev,
> libjansson4]
> + - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev,
> + libfdt-dev, libjansson-dev]
>
> _aarch64_packages: &aarch64_packages
> - *required_packages
> --
> 2.11.0
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
@@ -15,7 +15,7 @@ addons:
update: true
packages: &required_packages
- [libnuma-dev, python3-setuptools, python3-wheel, python3-pip, ninja-build]
- - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev, libjansson4]
+ - [libbsd-dev, libpcap-dev, libibverbs-dev, libcrypto++-dev, libfdt-dev, libjansson-dev]
_aarch64_packages: &aarch64_packages
- *required_packages