Re: [PATCH] libasm: Make libasm.h include work without relying on any other include.

[Mark Wielaard]

On Sun, 2020-07-05 at 00:17 +0200, Mark Wielaard wrote:
> The public headers should be usable when includes as is.
> libasm.h wasn't because it was using gelf.h data structures without
> include gelf.h. Include it now in libasm.h.
> 
> Add a new testcase run-test-includes.sh to test all public headers
> can be included "standalone".

Pushed to master

[PATCH] configure.ac: Set -DBAD_FTS=1 also for CXXFLAGS.

[Mark Wielaard]

debuginfod.cxx checks whether or not BAD_FTS is defined.
This only matters on old 32bit arches.

Signed-off-by: Mark Wielaard 
---
 ChangeLog| 4 
 configure.ac | 3 ++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 99f50492..c6b526fe 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2020-07-17  Mark Wielaard  
+
+   * configure.ac: Set -DBAD_FTS=1 also for CXXFLAGS.
+
 2020-06-19  Mark Wielaard  
 
* Makefile.am (SUBDIRS): Always add debuginfod.
diff --git a/configure.ac b/configure.ac
index 6f04d269..e2f213ab 100644
--- a/configure.ac
+++ b/configure.ac
@@ -261,7 +261,8 @@ dnl check it before including config.h (which might define 
_FILE_OFFSET_BITS).
 AC_CACHE_CHECK([whether fts.h is bad when included (with LFS)], ac_cv_bad_fts,
   [AC_COMPILE_IFELSE([AC_LANG_SOURCE([[#include ]])],
 ac_cv_bad_fts=no, ac_cv_bad_fts=yes)])
-AS_IF([test "x$ac_cv_bad_fts" = "xyes"], [CFLAGS="$CFLAGS -DBAD_FTS=1"])
+AS_IF([test "x$ac_cv_bad_fts" = "xyes"],
+  [CFLAGS="$CFLAGS -DBAD_FTS=1" CXXFLAGS="$CXXFLAGS -DBAD_FTS=1"])
 
 # See if we can add -D_FORTIFY_SOURCE=2. Don't do it if it is already
 # (differently) defined or if it generates warnings/errors because we
-- 
2.18.4