On Sun 11/02/2018 11:17, Landry Breuil wrote: > On Sun, Feb 11, 2018 at 10:47:16AM +0100, Landry Breuil wrote: > > On Sun, Feb 11, 2018 at 10:17:40AM +0100, Jeremie Courreges-Anglas wrote: > > > On Sun, Feb 11 2018, Landry Breuil <lan...@openbsd.org> wrote: > > > >> > > Of the 23 victims of devel/json-c I only tested net/isc-bind > > > >> > > (rndc-confgen to be more precise), which seems to work ok. It > > > >> > > would be > > > >> > > great if more people could check if this update works for them. > > > >> > > > > >> > I suppose that means "i've built all of them but only runtime-tested > > > >> > bind", right ? :) > > > >> > > > >> ...not exactly. Let me rephrase the last bit: > > > >> > > > >> I only built tested 2 victims, and runtime tested isc-bind. > > > > > > > > Right. In the past json-c has proven itself "interesting" when upgrading > > > > it, so i'll put this diff in my next bulk along cmake stuff. > > > > > > I have tried an upgrade to the 0.13 release, some of the structures are > > > made opaque, which either breaks consumers or forces them to implement > > > ugly hacks. So unless I missed something, upgrading to 0.13 is not > > > possible. > > > > It is definitely possible, json-c fucks up headers/structs at every > > upgrade, we're used to it. > > > > FreeBSD upgraded it already, cf > > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224675 & > > https://svnweb.freebsd.org/ports?view=revision&revision=457965 > > so the way is paved. Just need to cherrypick fixes/hacks there and > > there. Locally, i already know gdal and postgis fail to build with > > json-c 0.13. > > > > FreeBSD decided to keep installing the private header to fix things, > Cf > https://svnweb.freebsd.org/ports/head/devel/json-c/Makefile?r1=457965&r2=457964&pathrev=457965 > > An option is to do the same - Granted, this is not the clean way to go, > and hopefully over time consumers will stop including it directly. The > other options is to look at consumers and backport/cherrypick fixes. > > For example gdal was properly fixed in > https://github.com/OSGeo/gdal/pull/277, postgis in > https://github.com/postgis/postgis/pull/173 and both stopped poking at > internal struct members. > > Landry
I prefer fixing the consumers, as their upcoming releases support building with json-c 0.13. I have created diffs for: geo/gdal https://arcfour.nl/H8QhDVZ6Kv geo/postgis https://arcfour.nl/v2J0dq9V68 sysutils/syslog-ng https://arcfour.nl/hZGVidSDhy Actually, other people created diffs. I only applied, and build tested them. My intention is to mail the maintainers of the above and explain what I want to do. -- Björn Ketelaars GPG key: 0x4F0E5F21
