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

Libjson fix #36

Closed
wants to merge 2 commits into from
Closed

Libjson fix #36

wants to merge 2 commits into from

Conversation

mdlincoln
Copy link
Contributor

I have run into errors with your cmake file correctly locating libjsoncpp, both on OS X (10.10.5, with libjsoncpp installed via homebrew) as well as Ubuntu 14.04 (installed via apt-get). This has also been described in #30

# cmake ../
-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.8.4
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Looking for include file pthread.h
-- Looking for include file pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE  
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.26") 
-- Found libmicrohttpd: /usr/lib/x86_64-linux-gnu/libmicrohttpd.so  
-- Could NOT find libjsoncpp (missing:  LIBJSONCPP_INCLUDE_DIR) 
-- Configuring done
-- Generating done
-- Build files have been written to: /root/pastec/build

The following patch described in #8 fixes this for both OS X and Ubuntu. I have not yet tested it on other systems, however.

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.

1 participant