From: Junio C Hamano <gitster@pobox•com>
To: "Jiang Xin" <worldhello.net@gmail•com>,
"Torsten Bögershausen" <tboegi@web•de>,
"Jeff Hostetler" <jeffhostetler@github•com>
Cc: "Alexander Shopov" <ash@kambanaria•org>,
"Git List" <git@vger•kernel.org>,
jmas@softcatala•org, alexhenrie24@gmail•com,
ralf.thielow@gmail•com, matthias.ruester@gmail•com,
phillip.szelat@gmail•com, vyruss@hellug•gr,
christopher.diaz.riv@gmail•com, jn.avila@free•fr,
flashcode@flashtux•org, bagasdotme@gmail•com,
"Ævar Arnfjörð Bjarmason" <avarab@gmail•com>,
alessandro.menti@alessandromenti•it, elongbug@gmail•com,
cwryu@debian•org, uneedsihyeon@gmail•com, arek_koz@o2•pl,
dacs.git@brilhante•top, "insolor@gmail•com" <insolor@gmail•com>,
peter@softwolves•pp.se, bitigchi@me•com, ark@cho•red,
kate@kgthreads•com, "vnwildman@gmail•com" <vnwildman@gmail•com>,
pclouds@gmail•com, "dyroneteng@gmail•com" <dyroneteng@gmail•com>,
"oldsharp@gmail•com" <oldsharp@gmail•com>,
"lilydjwg@gmail•com" <lilydjwg@gmail•com>,
me@angyi•io, "pan93412@gmail•com" <pan93412@gmail•com>,
"franklin@goodhorse•idv.tw" <franklin@goodhorse•idv.tw>
Subject: Re: Is there any interest in localizing term delimiters in git messages?
Date: Thu, 19 Oct 2023 10:52:11 -0700 [thread overview]
Message-ID: <xmqqcyxaxzxw.fsf@gitster.g> (raw)
In-Reply-To: <CANYiYbEqTH975j9E0GTbSbexrw3MLhKwBCw7mibfnWbxZ+-_yw@mail.gmail.com> (Jiang Xin's message of "Thu, 19 Oct 2023 13:08:10 +0800")
Jiang Xin <worldhello.net@gmail•com> writes:
> I tried to find similar patterns in `po/bg.po` using:
>
> $ git grep -h -B5 '([a-zA-Z_\.]*_[a-zA-Z_\.]\+)' po/bg.po
>
> And find other translated variable names in Bulgarian as follows:
> ...
> I suppose it would be better to keep those variable names
> unchanged.
To me, all of them refer to names given to variables, functions, and
mechanisms used internally as implementation details, and they are
meant to help developers diagnose when end-users hit these errors.
I agree with you that translating these would be counter-productive
for that purpose.
Having said that, I have to wonder if in an ideal world these should
be written in terms that are more end-user facing.
> * cookie_result in builtin/fsmonitor--daemon.c:
>
> error(_("fsmonitor: cookie_result '%d' != SEEN"),
[jch: cc'ed JeffH for area expertise]
For example, what does it mean to the end user when the
cookie->result we retrieve is different from FCIR_SEEN? We lost
sync with the fsmonitor daemon backend and to avoid yielding
incorrect data we will be giving the "trivial" response only? It is
not obvious from the code and b05880d3 (fsmonitor--daemon: use a
cookie file to sync with file system, 2022-03-25) that added it why
the end-user might even want to be shown this message [*]. I wonder
if this should be an untranslated trace2_* message that are meant
for debugging.
Side note: and isn't the significance of the event
"warning", not "error"? As far as the end-user is
concerned, after emitting this message
Also some of them might better be a BUG(), instead of die(_()).
> * crlf_action in convert.c:
>
> warning(_("illegal crlf_action %d"), (int)crlf_action);
[jch: cc'ed Torsten for area expertise].
For example, can convert.c::output_eol() be called with an illegal
crlf_action that is not covered by the switch() statement due to
data error, not a programming error? From my quick scan, it looks
like that the error should never happen no matter what end-user
mistakes (e.g., misspelt attribute and configuration variable names
in their files) are fed to convert_attrs(), and can come only from a
bug in that function (e.g., long and convoluted if/else cascade fails
to assign any value to ca->crlf_action and leaves an undefined and
"illegal" value there).
Thanks.
next prev parent reply other threads:[~2023-10-19 17:58 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-17 21:09 Is there any interest in localizing term delimiters in git messages? Alexander Shopov
2023-10-17 21:49 ` Junio C Hamano
2023-10-18 2:01 ` Jiang Xin
2023-10-18 2:47 ` Junio C Hamano
2023-10-18 3:08 ` [PATCH] commit: do not use cryptic "new_index" in end-user facing messages Junio C Hamano
2023-10-19 5:08 ` Is there any interest in localizing term delimiters in git messages? Jiang Xin
2023-10-19 17:52 ` Junio C Hamano [this message]
2023-10-19 18:07 ` Jeff Hostetler
2023-10-19 19:18 ` Junio C Hamano
2023-10-19 19:47 ` Torsten Bögershausen
2023-10-19 20:16 ` Junio C Hamano
2023-10-21 9:30 ` Peter Krefting
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqcyxaxzxw.fsf@gitster.g \
--to=gitster@pobox$(echo .)com \
--cc=alessandro.menti@alessandromenti$(echo .)it \
--cc=alexhenrie24@gmail$(echo .)com \
--cc=arek_koz@o2$(echo .)pl \
--cc=ark@cho$(echo .)red \
--cc=ash@kambanaria$(echo .)org \
--cc=avarab@gmail$(echo .)com \
--cc=bagasdotme@gmail$(echo .)com \
--cc=bitigchi@me$(echo .)com \
--cc=christopher.diaz.riv@gmail$(echo .)com \
--cc=cwryu@debian$(echo .)org \
--cc=dacs.git@brilhante$(echo .)top \
--cc=dyroneteng@gmail$(echo .)com \
--cc=elongbug@gmail$(echo .)com \
--cc=flashcode@flashtux$(echo .)org \
--cc=franklin@goodhorse$(echo .)idv.tw \
--cc=git@vger$(echo .)kernel.org \
--cc=insolor@gmail$(echo .)com \
--cc=jeffhostetler@github$(echo .)com \
--cc=jmas@softcatala$(echo .)org \
--cc=jn.avila@free$(echo .)fr \
--cc=kate@kgthreads$(echo .)com \
--cc=lilydjwg@gmail$(echo .)com \
--cc=matthias.ruester@gmail$(echo .)com \
--cc=me@angyi$(echo .)io \
--cc=oldsharp@gmail$(echo .)com \
--cc=pan93412@gmail$(echo .)com \
--cc=pclouds@gmail$(echo .)com \
--cc=peter@softwolves$(echo .)pp.se \
--cc=phillip.szelat@gmail$(echo .)com \
--cc=ralf.thielow@gmail$(echo .)com \
--cc=tboegi@web$(echo .)de \
--cc=uneedsihyeon@gmail$(echo .)com \
--cc=vnwildman@gmail$(echo .)com \
--cc=vyruss@hellug$(echo .)gr \
--cc=worldhello.net@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox