Silo icon indicating copy to clipboard operation
Silo copied to clipboard

cmake: add json support

Open junghans opened this issue 2 months ago • 3 comments

junghans avatar Oct 24 '25 20:10 junghans

@markcmiller86 I got the cmake logic done, but json-c-0.10 is like 10+ year old and the more recent json-c-0.18 has a different includedir (json/ -> json-c/) and a slightly different library interface.

I would recommend to use json-0.18 or so and just use the distribution package instead of build it yourself.

junghans avatar Oct 24 '25 21:10 junghans

I would recommend to use json-0.18 or so and just use the distribution package instead of build it yourself.

Well, that is potentially problematic because what Silo is using is a enhanced json-c library with some features not available in the standard json release. I will give it some thought though.

markcmiller86 avatar Oct 26 '25 05:10 markcmiller86

I see, the best might be to hide that cmake option for the time being and figure this out after the release.

junghans avatar Oct 26 '25 17:10 junghans

#510 replaces this

markcmiller86 avatar Dec 04 '25 05:12 markcmiller86