Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Federico Vaga <federico.vaga@vaga.pv.it>
To:     Jonathan Corbet <corbet@lwn.net>, linux-doc@vger.kernel.org,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>
Cc:     Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] docs: it_IT: address invalid reference warnings
Date:   Sun, 07 Jun 2020 00:06:02 +0200
Message-ID: <11587824.O9o76ZdvQC@harkonnen>
In-Reply-To: <3060760.44csPzL39Z@harkonnen>
References: <20200531185618.7099-1-lukas.bulwahn@gmail.com> <3060760.44csPzL39Z@harkonnen>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

I re-read the documents with the full context.

Moving to the directive :doc: for only those two references (like I was=20
arguing in the previous email) will make the document inconsistent.
So, the patch is fine for me as it is.

I will finish and push the translation for ../core-api/symbol-namespace.rst
and move the link again

On Tuesday, June 2, 2020 10:37:21 AM CEST Federico Vaga wrote:
> On Sunday, May 31, 2020 8:56:18 PM CEST Lukas Bulwahn wrote:
> > Documentation generation warns:
> >   it_IT/kernel-hacking/hacking.rst:
> >     WARNING: unknown document: ../core-api/symbol/namespaces
> >  =20
> >   it_IT/process/5.Posting.rst:
> >     WARNING: undefined label: it_email_clients
> >  =20
> >   it_IT/process/submitting-patches.rst:
> >     WARNING: undefined label: it_email_clients
> >  =20
> >   it_IT/process/howto.rst:
> >      WARNING: undefined label: it_managementstyle
> >=20
> > Refer to English documentation, as Italian translation does not exist,
> > and
>=20
> The file exists! On my disk :D
> My mistake, I have an almost done translation for that and of course I do
> not see the warning.
>=20
> > add labels for Italian process documents to resolve label references.
>=20
> I think we have agreed to not use labels but instead to sue the directive
>=20
> :doc: instead. This fix should happen in the document that points here. W=
hen
> :I
> posted the new translations I removed those labels but forgot to fix:
> it_IT/process/5.Posting.rst, it_IT/process/submitting-patches.rst and it_=
IT/
> process/howto.rst
>=20
> :doc:`../process/email-clients`
> :doc:`../process/management-style`
>=20
> I should be more meticulous and regenerate the full translation every tim=
e.
> Lesson learned. Sorry for that and thanks
>=20
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> > ---
> > Jonathan, please pick this quick fix of warnings.
> >=20
> > applies on doc-next and next-20200529
> >=20
> >  Documentation/translations/it_IT/kernel-hacking/hacking.rst   | 4 ++--
> >  Documentation/translations/it_IT/process/email-clients.rst    | 2 ++
> >  Documentation/translations/it_IT/process/management-style.rst | 2 ++
> >  3 files changed, 6 insertions(+), 2 deletions(-)
> >=20
> > diff --git a/Documentation/translations/it_IT/kernel-hacking/hacking.rst
> > b/Documentation/translations/it_IT/kernel-hacking/hacking.rst index
> > 6aab27a8d323..e9a2e92134f0 100644
> > --- a/Documentation/translations/it_IT/kernel-hacking/hacking.rst
> > +++ b/Documentation/translations/it_IT/kernel-hacking/hacking.rst
> > @@ -634,7 +634,7 @@ Definita in ``include/linux/export.h``
> >=20
> >  Questa =E8 una variate di `EXPORT_SYMBOL()` che permette di specificar=
e uno
> >  spazio dei nomi. Lo spazio dei nomi =E8 documentato in
> >=20
> > -:doc:`../core-api/symbol-namespaces`
> > +:doc:`../../../core-api/symbol-namespaces`
> >=20
> >  :c:func:`EXPORT_SYMBOL_NS_GPL()`
> > =20
> >  --------------------------------
> >=20
> > @@ -643,7 +643,7 @@ Definita in ``include/linux/export.h``
> >=20
> >  Questa =E8 una variate di `EXPORT_SYMBOL_GPL()` che permette di specif=
icare
> >=20
> > uno spazio dei nomi. Lo spazio dei nomi =E8 documentato in
> > -:doc:`../core-api/symbol-namespaces`
> > +:doc:`../../../core-api/symbol-namespaces`
> >=20
> >  Procedure e convenzioni
> >  =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> >=20
> > diff --git a/Documentation/translations/it_IT/process/email-clients.rst
> > b/Documentation/translations/it_IT/process/email-clients.rst index
> > 89abf6d325f2..66d3d65776f7 100644
> > --- a/Documentation/translations/it_IT/process/email-clients.rst
> > +++ b/Documentation/translations/it_IT/process/email-clients.rst
> > @@ -3,6 +3,8 @@
> >=20
> >  :Original: :doc:`../../../process/email-clients`
> >  :Translator: Alessia Mantegazza <amantegazza@vaga.pv.it>
> >=20
> > +.. _it_email_clients:
> > +
> >=20
> >  Informazioni sui programmi di posta elettronica per Linux
> >  =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D
> >=20
> > diff --git a/Documentation/translations/it_IT/process/management-style.=
rst
> > b/Documentation/translations/it_IT/process/management-style.rst index
> > c709285138a7..76ed074082ea 100644
> > --- a/Documentation/translations/it_IT/process/management-style.rst
> > +++ b/Documentation/translations/it_IT/process/management-style.rst
> > @@ -3,6 +3,8 @@
> >=20
> >  :Original: :doc:`../../../process/management-style`
> >  :Translator: Alessia Mantegazza <amantegazza@vaga.pv.it>
> >=20
> > +.. _it_managementstyle:
> > +
> >=20
> >  Il modello di gestione del kernel Linux
> >  =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D