Re: [PATCH] docs/ja_JP: translate submitting-patches.rst (review and report tags)

[Akira Yokosawa <akiyks@gmail.com>]

Hi Kurita-san,

First of all, this email has a lot of bogus @www.redadmin.org addresses
in To: and Cc:.  Please don't do that.

On 9/3/26 11:03, Akiyoshi Kurita wrote:
> Translate the section describing Reported-by, Tested-by, Reviewed-by,
> Suggested-by, and Fixes tags into Japanese.
> 
> This includes the Reviewer's Statement and the guidance on adding or
> removing review and testing tags in subsequent patch versions.
> 
> Keep the Japanese submitting-patches documentation aligned with the
> corresponding English section.

Inline comments below.

> 
> Signed-off-by: Akiyoshi Kurita <weibu@redadmin.org>
> ---
>  .../ja_JP/process/submitting-patches.rst      | 91 +++++++++++++++++++
>  1 file changed, 91 insertions(+)
> 
> diff --git a/Documentation/translations/ja_JP/process/submitting-patches.rst b/Documentation/translations/ja_JP/process/submitting-patches.rst
> index 797b70b97fa7..0b4e22b015d7 100644
> --- a/Documentation/translations/ja_JP/process/submitting-patches.rst
> +++ b/Documentation/translations/ja_JP/process/submitting-patches.rst
> @@ -573,3 +573,94 @@ Acked-by:、Cc:、Co-developed-by: を使用する場合
>      Signed-off-by: From Author <from@author.example.org>
>      Co-developed-by: Submitting Co-Author <sub@coauthor.example.org>
>      Signed-off-by: Submitting Co-Author <sub@coauthor.example.org>
> +
> +
> +Reported-by:、Tested-by:、Reviewed-by:、Suggested-by:、Fixes: の使用
> +---------------------------------------------------------------------
> +
> +``Reported-by:`` タグは、バグを発見して報告した人の功績を示すものであり、
> +今後も再び協力してもらうきっかけになることが期待されます。このタグは
> +バグを対象としています。機能追加の要望に対する功績を示すためには
> +使用しないでください。報告が Web 上で参照できない場合を除き、
> +``Reported-by:`` タグの後には、その報告を指す ``Closes:`` タグを
> +付ける必要があります。パッチが報告された問題の一部だけを修正する場合は、

   付けてください。報告された問題の一部だけを修正するパッチの場合は、

> +``Closes:`` の代わりに ``Link:`` タグを使用できます。なお、
> +``Reported-by:`` は、名前を記載される本人の明示的な許可なしに
> +使用できる可能性がある三つのタグのうちの一つです。詳細については、
> +後述の「人をタグ付けするには許可が必要」を参照してください。
> +
> +``Tested-by:`` タグは、記載された人が（何らかの環境で）パッチを
> +正常にテストしたことを示します。このタグは、何らかのテストが
> +実施されたことをメンテナーに知らせ、将来のパッチでテスターを
> +探す手段を提供するとともに、テスターの功績を記録します。
> +
> +一方、``Reviewed-by:`` は、パッチがレビューされ、以下の
> +レビュー担当者の声明に従って受け入れ可能と判断されたことを示します::
> +
> +        レビュー担当者による確認声明

You used "レビューア" for "reviewer" in your earlier submissions.
Is there any reason for the different word/term ?

Also, "Reviewer's statement of oversight" is similar to the DCO.
I think it should be translated in a way so that it looks like
a legal statement.
Especially, I'd use "だ体", or formal style.

I would rather put the original English text here, and append the
translation as a 試訳 (or 訳例).

> +
> +        ``Reviewed-by:`` タグを付けることにより、私は以下を表明します:
> +
> +         (a) このパッチがメインラインカーネルへの取り込みに適切であり、
> +             その準備が整っているかを評価するため、技術的なレビューを
> +             行いました。
> +
> +         (b) パッチに関する問題、懸念、または質問はすべて提出者に
> +             伝えられています。私は、自分のコメントに対する提出者の
> +             回答に満足しています。
> +
> +         (c) この提出物には改善できる点が残っている可能性がありますが、
> +             現時点では、(1) カーネルに対する価値のある変更であり、
> +             (2) 取り込みに反対する理由となる既知の問題がないと
> +             考えています。
> +
> +         (d) 私はこのパッチをレビューし、妥当であると考えていますが、
> +             別途明示されていない限り、そのパッチが記載された目的を
> +             達成すること、またはいかなる状況でも正しく動作することを
> +             保証するものではありません。
> +
> +``Reviewed-by:`` タグは、そのパッチが重大な技術的問題を残していない、
> +カーネルに対する適切な変更であるという意見を表明するものです。
> +必要なレビューを行った既知の身元を持つ人であれば、関心のある
> +レビュー担当者は誰でもパッチに ``Reviewed-by:`` タグを付けられます。
> +このタグは、レビュー担当者の功績を示すとともに、どの程度のレビューが
> +行われたかをメンテナーに知らせる役割を果たします。対象分野を理解し、
> +十分なレビューを行うことで知られているレビュー担当者が付けた
> +``Reviewed-by:`` タグは、通常、パッチがカーネルに取り込まれる
> +可能性を高めます。
> +
> +``Tested-by:`` と ``Reviewed-by:`` の両方のタグは、メーリングリスト上で
> +テスターまたはレビュー担当者から受け取った後、次のバージョンを
> +送信するときに、作者が該当するパッチへ追加する必要があります。
> +ただし、次のバージョンでパッチが大幅に変更された場合は、これらのタグが
> +適用できなくなる可能性があるため、削除する必要があります。
> +通常、誰かの ``Acked-by:``、``Tested-by:``、または ``Reviewed-by:`` タグを
> +削除した場合は、その理由をパッチの変更履歴で説明する必要があります
> +（``---`` 区切りの後）。
> +
> +``Suggested-by:`` タグは、パッチのアイデアが記載された人によって
> +提案されたことを示し、そのアイデアに対する功績を記録します。
> +アイデアを提案した人の功績をきちんと記録すれば、今後も再び協力して
> +もらうきっかけになることが期待されます。なお、これは、名前を
> +記載される本人の明示的な許可なしに使用できる可能性がある三つの
> +タグのうちの一つです。詳細については、後述の
> +「人をタグ付けするには許可が必要」を参照してください。
> +
> +``Fixes:`` タグは、そのパッチが以前のコミットにあるバグを修正する
> +ことを示します。これは、問題がどこで発生したのかを容易に特定できる
> +ようにし、バグ修正のレビューに役立ちます。また、このタグは stable
> +kernel チームが、どの stable kernel バージョンに修正を適用すべきかを
> +判断する際にも役立ちます。パッチが修正するバグを示す方法としては、
> +これが推奨されています。詳細については :ref:`describe_changes` を

This will point the original (English) section.  File-local reference
can be made by:

                        詳細については `変更内容を記述する`_ を

> +参照してください。
> +
> +注意: ``Fixes:`` タグを付けても、stable kernel rules の手順や、
> +stable 向けパッチ候補すべてで ``Cc: stable@vger.kernel.org`` を
> +指定する要件がなくなるわけではありません。詳細については、
> +Documentation/process/stable-kernel-rules.rst を参照してください。
> +
> +最後に、タグを付けることは歓迎され、通常は非常にありがたいものですが、
> +署名者（すなわち、提出者およびメンテナー）は、提示されたタグを
> +適用するかどうかを自身の判断で決めることができます。

This label is not needed in this patch/file ...

> +
> +.. _tagging_people:

Thanks, Akira