Re: solaris iconv patch

Hi Again, First, Happy new year!! :) After reading some stuff, I must say that patch is not what we want, because with that patch I would have to run PHP with: LD_PRELOAD="/usr/local/lib/libiconv_plug.so" php So, I've cooked another patch, that really links to libiconv: http://mega.ist.utl.pt/~ncpl/php_libiconv_solaris.txt This patch also defines HAVE_LIBICONV when necessary, which it wasn't (thus breaking phpinfo()). Nuno ----- Original Message -----

Hi, I've made a little patch to fix iconv extension build problems on Solaris. The trick is to define LIBICONV_PLUG before loading the iconv.h header (so that is doens't define iconv* stuff as libiconv*). Patch: http://mega.ist.utl.pt/~ncpl/php_iconv_solaris.txt Nuno P.S.: there was some bug report about this, but I couldn't found the id