> > > AH_BOTTOM(
> > > #define N_(x) x
> > > +#define GETTEXT_NOOP(x)
> > > #ifdef ENABLE_NLS
> > > #include <locale.h>
> > > #include <libintl.h>
> > >
> >
> > Not sure if you realized this yet, but this change breaks the build on
> > Win32: http://www.mobsol.be/buildbot/win32-xp%20VS2005/builds/463/step-Build/0.
> > You define the GETTEXT_NOOP macro in configure.ac, which is not used
> > on Windows.
>
> what a mess! This was really not my intention. Please note that I
> published my patch on this list before I committed (OK, I waited only 1
> day). I also improved my older patch based on a vague suggestion on this
> list so I tried my best :-)
We know :-)
If you look at http://svn.collab.net/repos/svn/trunk/subversion/svn_private_config.hw
You'll see what you have to do in order to fix the Windows build.
(It's at the bottom of the file.)
> I search before for definitions of the similar macro N_ and found it
> only in configure.ac so I added the new macro definition there too.
> $ trunk> grep -r "define *N_" . | grep -v .svn
> ./configure.ac:#define N_(x) x
>
> Can you please provide a fix for Windows? Even better commit yourself.
> I do not have access to such a system (it's non-free).
>
> PS: At least the two new messages should be merged into PO files in
> 1.5.x as otherwise even for currently fully translated languages such
> as fr, pl these two will occur English in two important help messages.
HTH,
Erik.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-28 11:28:07 CEST