FS#48614 - [bind-tools] Include json.h from json-c instead of jsoncpp
Attached to Project:
Arch Linux
Opened by Dylon Edwards (deltaecho) - Wednesday, 16 March 2016, 21:37 GMT
Last edited by Sébastien Luttringer (seblu) - Sunday, 17 April 2016, 23:53 GMT
Opened by Dylon Edwards (deltaecho) - Wednesday, 16 March 2016, 21:37 GMT
Last edited by Sébastien Luttringer (seblu) - Sunday, 17 April 2016, 23:53 GMT
|
Details
Description:
In the source file, lib/isc/include/isc/json.h, there is an include at the top: #include <json/json.h> In the Arch ecosystem, this include points to the json.h header of jsoncpp rather than json-c, which breaks the build as follows: // BEGIN ERROR /usr/include/json/config.h:87:1: error: unknown type name 'namespace' namespace Json { ^ // END ERROR This file needs to be patched to include, json-c/json.h, instead: --- ./lib/isc/include/isc/json.h.orig 2016-03-16 14:26:45.123081502 -0700 +++ ./lib/isc/include/isc/json.h 2016-03-16 14:26:53.216428357 -0700 @@ -38,7 +38,7 @@ #include <json-c/json_object_iterator.h> #include <json-c/json_c_version.h> #else -#include <json/json.h> +#include <json-c/json.h> #endif #endif Additional info: * Package version: 9.10.3.P4-1 Steps to reproduce: * Attempt to execute makepkg against pkgver=9.10.3.P4-1 |
This task depends upon
Closed by Sébastien Luttringer (seblu)
Sunday, 17 April 2016, 23:53 GMT
Reason for closing: Works for me
Sunday, 17 April 2016, 23:53 GMT
Reason for closing: Works for me
Comment by
Sébastien Luttringer (seblu) -
Sunday, 17 April 2016, 23:53 GMT
build correctly in a chroot.