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

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to