[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Maposmatic-dev] [PATCH 1/2] [ocitysmap][i18n] Added two mo

From: Jeroen van Rijn
Subject: Re: [Maposmatic-dev] [PATCH 1/2] [ocitysmap][i18n] Added two more prefixes to Dutch i18n class
Date: Sun, 1 Apr 2012 11:20:43 +0200


My apologies, I've spotted a typo in my patch:
u"Ir.", u"Ingenieur", u"Ds.", u"Dominee", u"Deken",
- u"Drs.",
+ u"Drs.", u"Maj.", u"Majoor"
# counting words before street name,
# e.g. "1e Walstraat" => "Walstraat (1e)"

There's a missing comma after u"Majoor"

Which reminds me to take on that i18n test suite I'd originally planned some 18 months ago... things certainly can get hectic. :(

On Sun, Apr 1, 2012 at 11:05, Thomas Petazzoni <address@hidden> wrote:
Hello Jeroen!

Le Sun, 1 Apr 2012 10:24:01 +0200,
Jeroen van Rijn <address@hidden> a écrit :

> Signed-off-by: Jeroen van Rijn <address@hidden>
> ---
>  ocitysmap2/ |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks a lot, I have applied both of your patches, and they are
now working live on See:

Do you also intend to update the MapOSMatic translation? That would be

Also, next time, could you send your patches inline? It makes them
easier to apply while keeping your authorship. You can use git
send-email to do so.

Thanks again for your contributions and reactivity!

Thomas Petazzoni      
Logiciels Libres à Toulouse
Embedded Linux        


reply via email to

[Prev in Thread] Current Thread [Next in Thread]