Hi,

When porting/testing the D front-end to FreeBSD, I noticed that
backtrace supported returned false during the configuration of
libphobos.

The use of += assignment in the configure test was the reason why, and
now that's been corrected.

Bootstrapped and regression tested on x86_64-linux-gnu and x86_64-freebsd11.2.

Committed to trunk as r270377.

-- 
Iain
---
libphobos/ChangeLog:

2019-04-16  Iain Buclaw  <ibuc...@gdcproject.org>

        * config.h.in: Regenerate.
        * configure: Regenerate.
        * m4/druntime/libraries.m4 (DRUNTIME_LIBRARIES_BACKTRACE): Set
        CPPFLAGS correctly for backtrace support test.

---
diff --git a/libphobos/config.h.in b/libphobos/config.h.in
index 19266b3b5e4..0249849c890 100644
--- a/libphobos/config.h.in
+++ b/libphobos/config.h.in
@@ -54,3 +54,35 @@
 
 /* Define to 1 if you have the ANSI C header files. */
 #undef STDC_HEADERS
+
+/* Enable extensions on AIX 3, Interix.  */
+#ifndef _ALL_SOURCE
+# undef _ALL_SOURCE
+#endif
+/* Enable GNU extensions on systems that have them.  */
+#ifndef _GNU_SOURCE
+# undef _GNU_SOURCE
+#endif
+/* Enable threading extensions on Solaris.  */
+#ifndef _POSIX_PTHREAD_SEMANTICS
+# undef _POSIX_PTHREAD_SEMANTICS
+#endif
+/* Enable extensions on HP NonStop.  */
+#ifndef _TANDEM_SOURCE
+# undef _TANDEM_SOURCE
+#endif
+/* Enable general extensions on Solaris.  */
+#ifndef __EXTENSIONS__
+# undef __EXTENSIONS__
+#endif
+
+
+/* Define to 1 if on MINIX. */
+#undef _MINIX
+
+/* Define to 2 if the system does not provide POSIX.1 features except with
+   this defined. */
+#undef _POSIX_1_SOURCE
+
+/* Define to 1 if you need to in order for `stat' and other things to work. */
+#undef _POSIX_SOURCE
diff --git a/libphobos/configure b/libphobos/configure
index 87e4e4a7c9b..8079a73527d 100755
--- a/libphobos/configure
+++ b/libphobos/configure
@@ -14838,7 +14838,7 @@ fi
     LIBBACKTRACE=../../libbacktrace/libbacktrace.la
 
     gdc_save_CPPFLAGS=$CPPFLAGS
-    CPPFLAGS+=" -I../libbacktrace "
+    CPPFLAGS="$CPPFLAGS -I../libbacktrace "
 
     ac_fn_c_check_header_mongrel "$LINENO" "backtrace-supported.h" "ac_cv_header_backtrace_supported_h" "$ac_includes_default"
 if test "x$ac_cv_header_backtrace_supported_h" = xyes; then :
diff --git a/libphobos/m4/druntime/libraries.m4 b/libphobos/m4/druntime/libraries.m4
index 6e81fd99e4b..a7aab4dd88b 100644
--- a/libphobos/m4/druntime/libraries.m4
+++ b/libphobos/m4/druntime/libraries.m4
@@ -178,7 +178,7 @@ AC_DEFUN([DRUNTIME_LIBRARIES_BACKTRACE],
     LIBBACKTRACE=../../libbacktrace/libbacktrace.la
 
     gdc_save_CPPFLAGS=$CPPFLAGS
-    CPPFLAGS+=" -I../libbacktrace "
+    CPPFLAGS="$CPPFLAGS -I../libbacktrace "
 
     AC_CHECK_HEADER(backtrace-supported.h, have_libbacktrace_h=true,
       have_libbacktrace_h=false)

Reply via email to