Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace removed is_error macro #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jelly
Copy link

@jelly jelly commented Dec 27, 2017

The json-c library removed the is_error macro in 0.13, replaced the
macro calls with a != NULL statement.

The json-c library removed the is_error macro in 0.13, replaced the
macro calls with a != NULL statement.
@paulrho
Copy link
Contributor

paulrho commented Oct 13, 2018

This looks like it is required for the issue of the json build error, however, there may be another change required to bti.c:

-#include <json-c/json.h>
+#include <json/json.h>
...
-PKG_CHECK_MODULES([JSON], [json-c])
+PKG_CHECK_MODULES([JSON], [json])

At least, that's what seems to be altered in the Fedora build...investigating...

So far: seems maybe not - looks like we are ahead of the game (Package json-c changed json.h to json-c.h some time in the past)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants