Provide build-time switch to disable JSON support. #61
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This patch provides a mechanism to omit JSON support (and libjsmn dependency). By default, JSON support is included.
To build without JSON support, set the relevant CMAKE option with:
make CMAKE_OPTIONS=-DWITH_JSON=OFF
Ref: FLOWDM-693
Signed-off-by: David Antliff <[email protected]>
|
Retest this please |
1 similar comment
|
Retest this please |
…cmake. There is some difference between cmake 2.8.12 and 3.2.2 that results in LIBJSMN_INCLUDE_DIR being set to a null string by get_property(). This patch removes use of get_property in this case, ensuring that the variable has the correct value. Signed-off-by: David Antliff <[email protected]>
seank-img
pushed a commit
that referenced
this pull request
Mar 15, 2016
Provide build-time switch to disable JSON support.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch provides a mechanism to omit JSON support (and libjsmn dependency). By default, JSON support is included.
To build without JSON support, set the relevant CMAKE option with:
Ref: FLOWDM-693
Signed-off-by: David Antliff [email protected]