You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I had modify build process into your parser/builder - I had add (looks like a) full cmake integration:
After sudo make install library will be available for cmake's operator find_package (cjson REQUIRED) - no need write extra cmake-Finder's
Also, I am had merge two libraries (builder+parser) into one, not sure is it interesting for you.
Hm, your CMake script looks considerably more complex than #67, and I am not entirely convinced about the value of having CMake support for this library anyway. It is nearly always preferable to just add the single source file and single header file into one's own existing build system, is it not? This library is intentionally trivial to build, so it seems a little excessive to add more different build systems to it, no?
Hello!
I had modify build process into your parser/builder - I had add (looks like a) full cmake integration:
After
sudo make install
library will be available for cmake's operatorfind_package (cjson REQUIRED)
- no need write extra cmake-Finder'sAlso, I am had merge two libraries (builder+parser) into one, not sure is it interesting for you.
You can check it at:
https://github.com/vaclav2016/cjson
PS: Thanks! nice library !
The text was updated successfully, but these errors were encountered: