Re: Re: Iconv

Hello, I noticed by the recent cvs changes that iconv.c and config.m4 are now let to avoid PHP_ATOM approach, while php_iconv.h still adopts it. Is it really necessary? in acinclude.m4, PHP_SETUP_ICONV use PHP_DEFINE macro to make either HAVE_ICONV or HAVE_LIBICONV available. IMHO if php_iconv_supports_errno.h doesn't appear as a result of configure, there must be a build system problem, which has all but nothing to do with ext/iconv/*. Yasuo Ohgaki <yohgaki@ohgaki.net> wrote: > James Cox wrote: > > Joey (TML) pointed out to me that iconv is not currently building on head. > > > > Apparently iconv.c includes "php_iconv_supports_errno.h" which doesn't > > exist or get made at any point. > > > > Yasuo, this was your last commit here that broke it... > > > > -- James > > > > > James, > > Please add my email address to your TO list... > It will be fixed soon. > > -- > Yasuo Ohgaki > > > -- > PHP Development Mailing List <http://www.php.net/> > To unsubscribe, visit: http://www.php.net/unsub.php >