Re: plperl needs upgrade for Fedora 10

Поиск
Список
Период
Сортировка
От Andrew Dunstan
Тема Re: plperl needs upgrade for Fedora 10
Дата
Msg-id 49132FD3.2010904@dunslane.net
обсуждение исходный текст
Ответ на Re: plperl needs upgrade for Fedora 10  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: plperl needs upgrade for Fedora 10  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers

Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
>   
>> I'm thinking of something like this change to config/perl.m4:
>>     
>
>   
>> ! if test -z "$perl_embed_ldflags" ; then
>> !       AC_MSG_RESULT(no)
>> !       AC_MSG_ERROR([unable to determine flags to link embedded Perl])
>>     
>
> Hm, is it certain that "empty" is never a valid value for
> $perl_embed_ldflags?  

Yes. If it's empty we don't even link against libperl at all. That can't 
possibly be right.

> In any case I'm a bit confused how this fixes the
> problem --- it looks like the test is just relying on Config not Embed.
>
>             
>   

No, we get the ldopts from Embed and then *remove* the ccldflags from 
Config from that string. What is left is set as perl_embed_flags, and 
that's what mustn't be empty.

cheers

andrew




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: My review for the patch "Table command"
Следующее
От: Simon Riggs
Дата:
Сообщение: Re: Hot standby v5 patch assertion failure