On 02/01/2011 03:47 AM, Dave Page wrote:
On Tue, Feb 1, 2011 at 8:29 AM, Magnus Hagander<mag...@hagander.net>  wrote:
On Tue, Feb 1, 2011 at 09:08, Dave Page<dp...@pgadmin.org>  wrote:
On Tue, Feb 1, 2011 at 5:36 AM, Itagaki Takahiro
<itagaki.takah...@gmail.com>  wrote:
2011/1/27 Hiroshi Inoue<in...@tpf.co.jp>:
I see now the following lines in libintl.h of version
0.18.1.1 which didn't exist in 0.17 version.

The macro may cause a trouble especially on Windows.
Attached is a patch to disable the macro on Windows.
Can anyone test the fix?

I added the patch to the current commitfest for reminder.
https://commitfest.postgresql.org/action/patch_view?id=528
The QA team in EDB have tested the patch for me. It works as designed
and allows us to upgrade gettext to fix Japanese localisation on
Win64. Upgrading gettext without the patch will fix Japanese, but
break other translations (eg. French).
Do we need to backpatch this?
We've only seen the problem on 64 bit builds - but I guess it may
occur on 32 bit too, given the right version of gettext (they come
form different places for win32 vs. win64, so it's not entirely
straightforward to figure out).

So, it certainly needs to go to 9.0, and probably wouldn't hurt to put
it in 8.3/8.4 too.


Why are we only disabling the macro for WIN32 and not for the other platforms that the macro is defined for? Do we know it's not also a problem on Apple or Cygwin?

cheers

andrew

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to