Re: solaris iconv patch

From: Date: Sun, 01 Jan 2006 12:29:51 +0000
Subject: Re: solaris iconv patch
References: 1  Groups: php.internals 
Request: Send a blank email to [email protected] to get a copy of this message 
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

Thread (2 messages)

« previous php.internals (#21360) next »