git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Please pull gitk.git master branch

From
Paul Mackerras <paulus@samba.org>
Date
Dec 13, 2010, 21:03 UTC
Message-ID
<20101213210317.GA10027@brick.ozlabs.ibm.com>
In-Reply-To
<7vbp4q5ddo.fsf@alter.siamese.dyndns.org>
On Sun, Dec 12, 2010 at 11:40:51PM -0800, Junio C Hamano wrote:
Show 36 quoted lines
> Junio C Hamano <gitster@pobox.com> writes:
> 
> -- >8 --
> From: Junio C Hamano <gitster@pobox.com>
> Date: Sun, 12 Dec 2010 23:27:21 -0800
> Subject: [PATCH] Rename po/pt_BR.po to po/pt_br.po
> 
> The "msgfmt --tcl pt_BR.po" (at least on my box, GNU gettext 0.17) command
> generates pt_BR.msg, i.e. the country part gets downcased.  The resulting
> runtime (i.e. Tcl i18n) happily reads from pt_br.msg when run with the
> runtime locale set with LANG=pt_BR and/or LC_ALL=pt_BR so it seems to be
> the expected behaviour.
> 
> However, we seem to expect that the resulting file to be named pt_BR.msg,
> and try to generate and install it.
> 
> Currently our Makefile uses $(wildcard po/*.po) to grab the source PO
> files, expects them to produce $(subst .po,.msg,$(ALL_POFILES)), and its
> dependency rule is set to use "%.msg : %.po" pattern, all of which need
> to be adjusted with downcasing from po to msg files; the poor-man's msgfmt
> script also needs to learn the same downcasing.
> 
> Compared to that, renaming the input file to use lowercase countryname
> throughout the toolchain seems to be a lot cleaner solution to this
> glitch.
> 
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> ---
>  po/{pt_BR.po => pt_br.po} |    0
>  1 files changed, 0 insertions(+), 0 deletions(-)
>  rename po/{pt_BR.po => pt_br.po} (100%)
> 
> diff --git a/po/pt_BR.po b/po/pt_br.po
> similarity index 100%
> rename from po/pt_BR.po
> rename to po/pt_br.po

Sorry about the glitch. The renaming seems like the best idea, since all the other .po files have lowercase names. I was hoping to hear Alexandre Erwin Ittner's opinion, though.

If I apply this patch in my repo, is that going to cause problems in yours? Or, since the commit that adds this file is the head commit, I could just rewind it and reapply with the lowercase name. Do you see problems with that?

Paul.
Previous: Andreas SchwabNext: Junio C Hamano
Message 8 of 11 in “Please pull gitk.git master branch”
  1. Paul MackerrasDec 12, 2010
  2. Junio C HamanoDec 12, 2010
  3. Junio C HamanoDec 13, 2010
  4. Junio C HamanoDec 13, 2010
  5. Sverre RabbelierDec 13, 2010
  6. Andreas EricssonDec 13, 2010
  7. Andreas SchwabDec 13, 2010
  8. Paul MackerrasDec 13, 2010
  9. Junio C HamanoDec 13, 2010
  10. Paul MackerrasDec 14, 2010
  11. Junio C HamanoDec 14, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.