Dmitry Eremin-Solenikov(lumag) replied on github web page:

test/m4/miscellaneous.m4
line 5
@@ -4,5 +4,11 @@
 AC_ARG_ENABLE([test-cpp],
     [AS_HELP_STRING([--disable-test-cpp], [run basic test aginast cpp])],
     [test_cpp=$enableval],
-    [test_cpp=yes])
+    [AC_LANG_PUSH([C++])


Comment:
Missing AC_PROG_CXX.

Also I think it's better to write it in the following way:
```autoconf
AC_ARG_ENABLE([.......],
    [test_cpp=$enableval],
    [test_cpp=check])

if test "x$test_cpp" != "xno" ; then
    AC_PROG_CXX
    AC_CACHE_CHECK([if C++ compiler works], [odp_cv_cxx_works],
      [AC_COMPILE_IFELSE([....], [odp_cv_cxx_works=yes], 
[odp_cv_cxx_works=no])])
   # or you can rewrite this as two inline diffs
    AS_IF([test "x$test_cpp$odp_cv_cxx_works" = "xyesno"], 
[AC_MSG_FAILURE([...])],
               [test "x$test_cpp$odp_cv_cxx_works" = "xcheckno"], 
[AC_MSG_NOTICE([disabling C++ test])])
fi

```

https://github.com/Linaro/odp/pull/349#discussion_r157808329
updated_at 2017-12-19 16:39:58

Reply via email to