[libvirt] [PATCH] travis: Install Jansson instead of yajl

Andrea Bolognani posted 1 patch 5 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180515160301.26431-1-abologna@redhat.com
Test syntax-check passed
.travis.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[libvirt] [PATCH] travis: Install Jansson instead of yajl
Posted by Andrea Bolognani 5 years, 10 months ago
libvirt has switched to Jansson as its JSON implementation,
so we need it available for builds to succeed. yajl, on the
other hand, is no longer needed.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
Should be pushed at the same time as Ján's series.

 .travis.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 140072b801..c3ea9f73b9 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -24,7 +24,7 @@ matrix:
         - brew uninstall python mercurial postgis sfcgal cgal gdal
         - brew update
         - brew upgrade
-        - brew install python rpcgen yajl
+        - brew install python rpcgen jansson
       script:
         # We can't run make distcheck/syntax-check because they
         # fail on macOS, but doing 'install' and 'dist' gives us
@@ -61,6 +61,7 @@ addons:
       - libdevmapper-dev
       - libfuse-dev
       - libgnutls-dev
+      - libjansson-dev
       - libnetcf-dev
       - libnl-3-dev
       - libnl-route-3-dev
@@ -82,7 +83,6 @@ addons:
       - libxen-dev
       - libxml2-dev
       - libxml2-utils
-      - libyajl-dev
       - lvm2
       - make
 #     - nfs-common    # broken on trusty since build #807 (2018-01-09)
-- 
2.17.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] travis: Install Jansson instead of yajl
Posted by Daniel P. Berrangé 5 years, 10 months ago
On Tue, May 15, 2018 at 06:03:01PM +0200, Andrea Bolognani wrote:
> libvirt has switched to Jansson as its JSON implementation,
> so we need it available for builds to succeed. yajl, on the
> other hand, is no longer needed.
> 
> Signed-off-by: Andrea Bolognani <abologna@redhat.com>
> ---
> Should be pushed at the same time as Ján's series.
> 
>  .travis.yml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list