On Wed, Jul 20, 2011 at 18:35, Rainer Orth <r...@cebitec.uni-bielefeld.de> wrote: > I've hacked around this by wrapping the AM_ICONV calls in > AC_LANG_{PUSH, POP}(C++), but I think this exposes a fundamental > issue: the configure tests must be performed with the compiler used > for the build. That this works without is pure luck IMO.
Right, but it also applies to more than this test. If you wrapped in ifs more than just this call, the approach may be fine, but first I would like to look at Autoconf (or at the diff for the regenerated configure) to check that what you're doing is safe. I'm afraid it may not be, which means you're patch is not good for the configure part. :( > * Also, the definition of HAVE_DESIGNATED_INITIALIZERS was wrong for g++ > on Solaris which again defines __STDC_VERSION__ 199901L. To fix this, > I never define H_D_I if __cplusplus. Is this a valid definition of __STDC_VERSION__ at all? Paolo