[v2] ci: fix telemetry dependency in Travis
Checks
Commit Message
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 Linkeš <juraj.linkes@pantheon.tech> writes:
> libjansson4 is not enough to build telemetry. Replace it with
> libjansson-dev.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
Acked-by: Aaron Conole <aconole@redhat.com>
Hello,
On Wed, Apr 8, 2020 at 2:49 PM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:
>
> libjansson4 is not enough to build telemetry. Replace it with
> libjansson-dev.
Thanks for the patch Juraj.
Just a little comment, for the v2, there was no significant change and
you could have kept Aaron and Ruifeng review tags.
I just want others' opinion, since there is a work on removing the
dependency on libjansson for telemetry.
This current patch is a fix for the CI, that could be worth
backporting to 19.11.
Hence I am for taking it in master.
Then when the telemetry rework is merged, this rework will remove the
dependency.
On Thu, Apr 09, 2020 at 10:00:51AM +0200, David Marchand wrote:
> Hello,
>
> On Wed, Apr 8, 2020 at 2:49 PM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:
> >
> > libjansson4 is not enough to build telemetry. Replace it with
> > libjansson-dev.
>
> Thanks for the patch Juraj.
> Just a little comment, for the v2, there was no significant change and
> you could have kept Aaron and Ruifeng review tags.
>
>
> I just want others' opinion, since there is a work on removing the
> dependency on libjansson for telemetry.
>
That would be an end-goal but we are not fully there yet. Even with the
rework some parts of the library for legacy compatibility still depend on
jansson, though the majority of the code will compile without it. Therefore
this dependency should be kept in the CI beyond this release, as we weren't
planning on doing the additional work to completely replace it - though we
can re-evaluate that if people think that it's important to remove the
dependency completely.
/Bruce
On Thu, Apr 9, 2020 at 11:24 AM Bruce Richardson
<bruce.richardson@intel.com> wrote:
> On Thu, Apr 09, 2020 at 10:00:51AM +0200, David Marchand wrote:
> > I just want others' opinion, since there is a work on removing the
> > dependency on libjansson for telemetry.
> >
> That would be an end-goal but we are not fully there yet. Even with the
> rework some parts of the library for legacy compatibility still depend on
> jansson, though the majority of the code will compile without it. Therefore
> this dependency should be kept in the CI beyond this release, as we weren't
> planning on doing the additional work to completely replace it - though we
> can re-evaluate that if people think that it's important to remove the
> dependency completely.
Ok, thanks for the clarification Bruce.
I will go with this patch.
On Wed, Apr 8, 2020 at 2:49 PM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:
> libjansson4 is not enough to build telemetry. Replace it with
> libjansson-dev.
Fixes: 99889bd85228 ("ci: introduce Travis builds for GitHub repositories")
Cc: stable@dpdk.org
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
Acked-by: Aaron Conole <aconole@redhat.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
Applied, thanks.
@@ -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