Re: [PATCH] libmpx: remove AC_PROG_CXX/GCC_LIBSTDCXX_RAW_CXX_FLAGS from configure.ac

2015-04-27 Thread Ilya Enkovich 
2015-04-23 12:53 GMT+03:00 Steven Noonan ste...@uplinklabs.net:
 This causes libmpx to fail the configure stage on my build hosts because 
 'xg++'
 was invoked with the bogus '-funconfigured-libstdc++-v3' flag, and the 
 fallback
 preprocessor '/lib/cpp' did not exist on my systems.

 Since libmpx has no C++ code, requiring a C++ preprocessor and compiler at
 configure time doesn't make any sense. Simply remove the directives from
 configure.ac.

 I tested this change on my build environments and the libmpx configure 
 failures
 I was seeing went away. Recommend this be applied to trunk and the gcc-5
 branch.

Thank you for the patch.  I applied it to trunk and will apply to
gcc-5 branch later.

Ilya


 ChangeLog

 2015-04-23  Steven Noonan ste...@uplinklabs.net

* configure.ac: Drop AC_PROG_CXX and friends, since libmpx has no C++
  sources.
* configure: Regenerate.
 ---
  libmpx/configure.ac | 2 --
  1 file changed, 2 deletions(-)

 diff --git a/libmpx/configure.ac b/libmpx/configure.ac
 index 3f8b50f..463e855 100644
 --- a/libmpx/configure.ac
 +++ b/libmpx/configure.ac
 @@ -23,7 +23,6 @@ AC_MSG_RESULT($version_specific_libs)
  AC_CANONICAL_SYSTEM
  target_alias=${target_alias-$host_alias}
  AC_SUBST(target_alias)
 -GCC_LIBSTDCXX_RAW_CXX_FLAGS

  # See if supported.
  unset LIBMPX_SUPPORTED
 @@ -94,7 +93,6 @@ AC_SUBST(toolexeclibdir)
  m4_rename([_AC_ARG_VAR_PRECIOUS],[real_PRECIOUS])
  m4_define([_AC_ARG_VAR_PRECIOUS],[])
  AC_PROG_CC
 -AC_PROG_CXX
  m4_rename_force([real_PRECIOUS],[_AC_ARG_VAR_PRECIOUS])

  AM_PROG_CC_C_O
 --
 2.3.6

[PATCH] libmpx: remove AC_PROG_CXX/GCC_LIBSTDCXX_RAW_CXX_FLAGS from configure.ac

2015-04-23 Thread Steven Noonan 
This causes libmpx to fail the configure stage on my build hosts because 'xg++'
was invoked with the bogus '-funconfigured-libstdc++-v3' flag, and the fallback
preprocessor '/lib/cpp' did not exist on my systems.

Since libmpx has no C++ code, requiring a C++ preprocessor and compiler at
configure time doesn't make any sense. Simply remove the directives from
configure.ac.

I tested this change on my build environments and the libmpx configure failures
I was seeing went away. Recommend this be applied to trunk and the gcc-5
branch.

ChangeLog

2015-04-23  Steven Noonan ste...@uplinklabs.net

   * configure.ac: Drop AC_PROG_CXX and friends, since libmpx has no C++
 sources.
   * configure: Regenerate.
---
 libmpx/configure.ac | 2 --
 1 file changed, 2 deletions(-)

diff --git a/libmpx/configure.ac b/libmpx/configure.ac
index 3f8b50f..463e855 100644
--- a/libmpx/configure.ac
+++ b/libmpx/configure.ac
@@ -23,7 +23,6 @@ AC_MSG_RESULT($version_specific_libs)
 AC_CANONICAL_SYSTEM
 target_alias=${target_alias-$host_alias}
 AC_SUBST(target_alias)
-GCC_LIBSTDCXX_RAW_CXX_FLAGS
 
 # See if supported.
 unset LIBMPX_SUPPORTED
@@ -94,7 +93,6 @@ AC_SUBST(toolexeclibdir)
 m4_rename([_AC_ARG_VAR_PRECIOUS],[real_PRECIOUS])
 m4_define([_AC_ARG_VAR_PRECIOUS],[])
 AC_PROG_CC
-AC_PROG_CXX
 m4_rename_force([real_PRECIOUS],[_AC_ARG_VAR_PRECIOUS])
 
 AM_PROG_CC_C_O
-- 
2.3.6