Re: [PATCHES] snprintf() argument reordering not working

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, ntufar(at)gmail(dot)com, devrim(at)kivi(dot)com(dot)tr, mha(at)sollentuna(dot)net, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] snprintf() argument reordering not working
Date: 2005-12-06 04:23:53
Message-ID: 43951259.7010803@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:

>Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>
>>>OK, eyeball this for the REL8_1_STABLE branch, please. It seems to
>>>work for me. No exports necessary.
>>>
>>>
>
>
>
>>er try this instead. I missed a line from configure.in
>>
>>
>
>I cleaned this up slightly and committed it into HEAD and 8.1 branches.
>It appears to work in that I can force pgac_need_repl_snprintf to "yes"
>on a Linux machine and get a working build. But we need to verify that
>things are OK on Windows, both with the old libintl that the installer
>is using and with current libintl. Please build and test ...
>
>

the cleanup seems to have omitted the change I had to
src/interfaces/ecpg/pgtypeslib/extern.h, which causes a build failure - see

http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=loris&dt=2005-12-06%2003:30:36

If we don't do this then we need to link snprintf into libpgtypes just
like we do for ecpg, but it seems like overkill.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2005-12-06 04:39:44 Re: [PATCHES] snprintf() argument reordering not working
Previous Message Christopher Kings-Lynne 2005-12-06 03:58:39 Bug in pg_dump -c with casts

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-12-06 04:39:44 Re: [PATCHES] snprintf() argument reordering not working
Previous Message Tom Lane 2005-12-06 03:46:18 Re: [PATCHES] snprintf() argument reordering not working