Skip to content

Commit

Permalink
slingshot_metrics: new header include needed by shs-2.1.0
Browse files Browse the repository at this point in the history
libcxi.h fails to include cassini_cntr_desc.h, which is almost certainly
a bug in their header include tree. We include it explicitly until they
fix things.
  • Loading branch information
morrone authored and tom95858 committed Feb 1, 2024
1 parent 1c7a8db commit e33463b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 2 additions & 1 deletion configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -881,14 +881,15 @@ AM_CONDITIONAL([SYSCONFDIR_NOT_ETC], [test "${sysconfdir}" != "/etc"])
AC_LIB_HAVE_LINKFLAGS([cxi], [], [
#include <stddef.h> /* libcxi.h fails to include this */
#include <libcxi/libcxi.h>
#include <cassini_cntr_desc.h> /* needed at least starting with shs-2.1.0 */
])
AM_CONDITIONAL([HAVE_LIBCXI], [test "x$HAVE_LIBCXI" = xyes])

AC_ARG_ENABLE([slingshot],
[AS_HELP_STRING([--enable-slingshot], [require the slinghost related plugins @<:@default=check@:>@])],
[],
[enable_slingshot="check"])
AM_CONDITIONAL([ENABLE_SLINGSHOT], [test "x$enable_slingshot" != xno])
AM_CONDITIONAL([ENABLE_SLINGSHOT], [test "x$enable_slingshot" != xno -a "x$HAVE_LIBCXI" = xyes])
AS_IF([test "x$enable_slingshot" = xyes],[
AS_IF([test "x$HAVE_LIBCXI" = xno],
[AC_MSG_ERROR([libcxi or its headers not found])])
Expand Down
1 change: 1 addition & 0 deletions ldms/src/sampler/slingshot_metrics/slingshot_metrics.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@

#include <stddef.h> /* libcxi.h neglects to include this */
#include <libcxi/libcxi.h>
#include <cassini_cntr_desc.h> /* needed at least starting with shs-2.1.0 */
#define _GNU_SOURCE

#define SAMP "slingshot_metrics"
Expand Down

0 comments on commit e33463b

Please sign in to comment.