[PATCH 0 of 5] i18n patches

Matt Mackall mpm at selenic.com
Fri Jan 9 19:46:52 CST 2009


On Sat, 2009-01-10 at 01:26 +0100, Martin Geisler wrote:
> Hi everybody,
> 
> Another go of the i18n-patches :-)
> 
> * Patch 1 makes it easier to find the hg.mo files. The gettext module
>   would lookup the hg.mo file corresponding to xx.po in

Seems reasonable.

> * Patch 2 ensures that the output is encoded in the user's local
>   encoding (as given by HGENCODING, LANG, LC_ALL, ...).

This one puzzles me. Shouldn't we instead be choosing a locale file
based on LANG and LC_CTYPE? It seems that if this were the right thing
to do, gettext would be doing it for us as everything using it would
need it.

> * Patch 3 adds new targets to the Makefile for updating the hg.pot (PO
>   template) file and to generate xx.mo files. I've added these files
>   to the .hgignore file.

Is there any reason I'd ever run update-mo directly rather than just
'make'? If not, then we needn't add help for it.

> * Patch 4 adds a rough Danish translation.

Looks like this lives in po/, which seems like an unfortunate directory
name. Can we come up with something better?

> * Patch 5 adds a test case. One *must* run 'make update-mo' after
>   applying this patch, otherwise the test case will fail. The problem
>   is that run-tests.py use the setup.py script to install, and that
>   script does not know anything about generating the .mo files.

Oh, look, an answer to my question. Yeah, we want run-tests.py to just
work. So we can either hack setup.py or run-tests.py

>   I could go in and hack run-tests.py and make it run the Makefile.
>   That would add 'make' and 'msgfmt' to the required_tools in
>   run-tests.py, maybe making it even harder to port the tests.
> 
>   I could also skip the test if no translation is found.

Probably. It's not really important that we test a particular
translation, just that _() works.

Let's figure out the directory name issue, then get patches 1, 3, and 4
in. Then we can figure out what the right answers for 2 and 5 are, ok?

-- 
Mathematics is the supreme nostalgia of our time.



More information about the Mercurial-devel mailing list