Benjamin Kosnik - [v3] AM_ICONV to native-only (original) (raw)

This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.

• From: Benjamin Kosnik
• To: libstdc++ at gcc dot gnu dot org
• Cc: gcc-patches at gcc dot gnu dot org
• Date: Wed, 14 Feb 2007 12:15:17 +0100
• Subject: [v3] AM_ICONV to native-only

Thanks to Gaby, I was finally able to reproduce this issue by using powerpc-eabi as the target and removing --with-newlib flags.I am unsure as to reasons behind the variance in configure behavior between powerpc-eabi, arm-elf, and m32c-elf, with and without --with-newlib flags. Some work without this change, some do not.However, the enclosed patch fixes the configure fail in all cases, by moving AM_ICONV to native only.

tested x86/linux tested x86/linux x powerpc-eabi (without --with-newlib)

-benjamin

2007-02-14 Benjamin Kosnik bkoz@redhat.com

* configure.ac: Move AM_ICONV to native-only, add HAVE_ICONV for
newlib.
* configure: Regenerate.

Index: configure.ac

--- configure.ac (revision 121946) +++ configure.ac (working copy) @@ -86,7 +86,6 @@ AM_PROG_LIBTOOL AC_SUBST(enable_shared) AC_SUBST(enable_static) -AM_ICONV

Possibly disable most of the library.

TODO: Consider skipping unncessary tests altogether in this case, rather

@@ -182,6 +181,9 @@

AC_FUNC_MMAP

• For iconv support.

• AM_ICONV

• else

  This lets us hard-code the functionality we know we'll have in the cross

@@ -251,6 +253,8 @@

 AC_DEFINE(HAVE_S_ISREG)
 AC_DEFINE(HAVE_S_IFREG)

• AC_DEFINE(HAVE_ICONV)

else GLIBCXX_CROSSCONFIG fi

• Follow-Ups:
  • Re: [v3] AM_ICONV to native-only
    * From: Mark Mitchell