linux-kernel - Re: [PATCH v2 17/19] docs: replace transation references for reporting-bugs.rst

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20210407113934.1a61ad9e@coco.lan>
Date:   Wed, 7 Apr 2021 11:39:34 +0200
From:   Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
To:     Thorsten Leemhuis <linux@...mhuis.info>
Cc:     Linux Doc Mailing List <linux-doc@...r.kernel.org>,
        "Alexander A. Klimov" <grandmaster@...klimov.de>,
        Jonathan Corbet <corbet@....net>, Alex Shi <alexs@...nel.org>,
        Federico Vaga <federico.vaga@...a.pv.it>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Harry Wei <harryxiyou@...il.com>,
        Wu XiangCheng <bobwxc@...il.cn>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 17/19] docs: replace transation references for
 reporting-bugs.rst

Em Wed, 7 Apr 2021 10:52:14 +0200
Thorsten Leemhuis <linux@...mhuis.info> escreveu:

> On 07.04.21 10:20, Mauro Carvalho Chehab wrote:
> > Changeset d2ce285378b0 ("docs: make reporting-issues.rst official and delete reporting-bugs.rst")
> > dropped reporting-bugs.rst, in favor of reporting-issues.rst, but
> > translations still need to be updated, in order to point to the
> > new file.
> > 
> > Fixes: d2ce285378b0 ("docs: make reporting-issues.rst official and delete reporting-bugs.rst")
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>  
> 
> Well, yeah, might be the right thing to do. But FWIW: when I recently
> submitted the change that became d2ce285378b0 I actually pointed out
> that it breaks some of the translations. Back then I considered to do
> what you did with this patch, but among others got a reply from Jonathan
> who said "let the translators catch up on their own time". For details
> see this thread:
> https://lore.kernel.org/linux-doc/87h7krksvu.fsf@meer.lwn.net/

Hmm... at the e-mail you mentioned, Jon commented that:

	"None of the broken references actually generate warnings"

That's actually not the case: they do generate warnings if
the Kernel is built with CONFIG_WARN_MISSING_DOCUMENTS:

	Documentation/translations/zh_CN/admin-guide/reporting-issues.rst: Documentation/admin-guide/reporting-bugs.rst
	Documentation/translations/zh_CN/admin-guide/reporting-issues.rst: Documentation/admin-guide/reporting-bugs.rst

As it will call the ./scripts/documentation-file-ref-check.

That's basically why I detected and submitted a fix ;-)

Thanks,
Mauro