Skip to content

Compile issue #659

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

Closed
SloCompTech opened this issue Sep 2, 2017 · 4 comments
Closed

Compile issue #659

SloCompTech opened this issue Sep 2, 2017 · 4 comments
Assignees

Comments

@SloCompTech
Copy link
Contributor

Hi,
I installed jsoncpp via meson and ninja, all worked fine, until I tried to comple my program, then I got

In file included from /usr/local/include/json/autolink.h:9:0,
                 from /usr/local/include/json/json.h:9,
                 from Test.cpp:1:
/usr/local/include/json/config.h:146:32: error: #if with no expression
/usr/local/include/json/config.h:172:32: error: #if with no expression
In file included from /usr/local/include/json/json.h:10:0,
                 from Test.cpp:1:
/usr/local/include/json/value.h:354:32: error: #if with no expression

because #if insead of #ifdef in some header file.
I will prepare fix, and pull request.

@SloCompTech
Copy link
Contributor Author

Here is meson output

The Meson build system
Version: 0.42.0
Source dir: /home/martin/Workspace/Projects/Libraries/jsoncpp
Build dir: /home/martin/Workspace/Projects/Libraries/jsoncpp/build
Build type: native build
Project name: jsoncpp
Native C++ compiler: c++ (gcc 5.4.0)
Build machine cpu family: x86_64
Build machine cpu: x86_64
Configuring version.h using configuration
WARNING: The variable(s) 'JSONCPP_USE_SECURE_MEMORY' in the input file 'src/lib_json/version.h.in' are not present in the given configuration data
Build targets in project: 3

@lachlansimpson
Copy link

Can you roll 1.8.4 to include this patch please?

@cdunn2001
Copy link
Contributor

Ok. I'd like to get another fix in there too, but we have to wait for TravisCI to start working again. Next week, hopefully.

@cdunn2001 cdunn2001 self-assigned this Dec 18, 2017
@cdunn2001 cdunn2001 reopened this Dec 18, 2017
@cdunn2001
Copy link
Contributor

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

No branches or pull requests

3 participants