Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:21360
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Message-ID: <002301c60ecf$105828b0$0100a8c0@pc07653>
To: "Nuno Lopes" <nlopess@php.net>,
	"PHPdev" <internals@lists.php.net>
Cc: <sniper@php.net>
References: <006001c60d46$996df5b0$0100a8c0@pc07653>
Date: Sun, 1 Jan 2006 12:29:51 -0000
MIME-Version: 1.0
Content-Type: text/plain;
	format=flowed;
	charset="Windows-1252";
	reply-type=response
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] solaris iconv patch
From: nlopess@php.net ("Nuno Lopes")

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