Skip to content

Commit

Permalink
Refector JSON parsing to use libjansson
Browse files Browse the repository at this point in the history
The libjansson parser has better documentation
and better error reporting. The API is easier
to use resulting in less code required for
JSON parsing.

The __decomp decorator in front of most plugin
symbols has also been removed as this is
unnecessary for static symbols and variables.
  • Loading branch information
tom95858 committed May 31, 2024
1 parent 78faded commit c1d13d0
Show file tree
Hide file tree
Showing 11 changed files with 890 additions and 1,098 deletions.
2 changes: 2 additions & 0 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,8 @@ AC_C_FLEXIBLE_ARRAY_MEMBER
OVIS_PKGLIBDIR

AC_LIB_HAVE_LINKFLAGS([jansson], [], [#include <jansson.h>])
AS_IF([test "x$HAVE_LIBJANSSON" = xno],
[AC_MSG_ERROR([libjansson or jansson.h not found])])
AM_CONDITIONAL([HAVE_LIBJANSSON], [test "x$HAVE_LIBJANSSON" = xyes])

dnl change sharedstatedir default
Expand Down
2 changes: 1 addition & 1 deletion ldms/src/decomp/as_is/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ DECOMP_LIBADD = ../../core/libldms.la \
../../ldmsd/libldmsd_request.la

libdecomp_as_is_la_SOURCES = decomp_as_is.c
libdecomp_as_is_la_LIBADD = $(DECOMP_LIBADD)
libdecomp_as_is_la_LIBADD = $(DECOMP_LIBADD) $(LTLIBJANSSON)
pkglib_LTLIBRARIES += libdecomp_as_is.la
Loading

0 comments on commit c1d13d0

Please sign in to comment.