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
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Sébastien Luttringer (seblu)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

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
Comment by Sébastien Luttringer (seblu) - Sunday, 17 April 2016, 23:53 GMT
build correctly in a chroot.

Loading...