Hi Ian, Ian Beckwith wrote: > iconv fails the test suite due to a missing dep.
Thanks. This regression was introduced on 2010-11-26. > Trivial patch attached. Actually, the intent was that the 'iconv' module does not require the 'iconv-h' module. The AM_ICONV macro is documented (in the gettext documentation) to be usable on its own, and I find it good if testing the 'iconv' module also tests that AM_ICONV can be used standalone. I'm therefore applying this different fix: 2010-12-19 Bruno Haible <br...@clisp.org> iconv: Make it possible again to use this module without 'iconv-h'. * modules/iconv (configure.ac): Don't invoke gl_ICONV_MODULE_INDICATOR if it is not defined. Reported by Ian Beckwith <i...@erislabs.net>. --- modules/iconv.orig Sun Dec 19 11:09:36 2010 +++ modules/iconv Sun Dec 19 11:06:43 2010 @@ -9,7 +9,8 @@ configure.ac: AM_ICONV -gl_ICONV_MODULE_INDICATOR([iconv]) +m4_ifdef([gl_ICONV_MODULE_INDICATOR], + [gl_ICONV_MODULE_INDICATOR([iconv])]) Makefile.am: