forgot to copy the list on this response
---------- Forwarded message ----------
From: jerry gay <[EMAIL PROTECTED]>
Date: Aug 18, 2005 8:05 AM
Subject: Re: Is Configure.pl missing ICU ?
To: Adrian Lambeck <[EMAIL PROTECTED]>
On 8/18/05, Adrian Lambeck <[EMAIL PROTECTED]> wrote:
> Since nobody responded to my question up to now I had a look at the parrot
> source. Just to remind you what I was asking:
> Eventough parrot is compiled with icu support this does not show up when I
> query parrot for it (i.e. # /usr/lib/parrot-0.2.3/parrot parrot-config.imc
> libs).
>
> So I looked at config/auto/gmp.pl to compare it against config/gen/icu.pl
> gmp.pl has a line:
> Configure::Data->add(' ', 'libs', '-lgmp');
> but in icu.pl the line is:
> Configure::Data->add(' ', ccflags => "-I $icuheaders");
>
> So the parameter for 'libs' is not set eventhough I think it should be.
>
> Has this just been forgotten or is there a real reason against adding
> "-licuuc" (besides breaking mod_parrot) ?
>
it seems to me that the icu libs should indeed be added to the config.
i don't compile with icu, so i'm probably not the best one to test the
change--but if nobody takes ownership of this issue in the next few
days, i'll take a look at it.
~jerry
