Re: [PATCH] doc: fix reference to core-api/namespaces.rst

From: Jonathan Corbet
Date: Fri Nov 22 2019 - 12:34:41 EST


On Fri, 22 Nov 2019 12:53:37 +0100
Federico Vaga <federico.vaga@xxxxxxxxxx> wrote:

> This patch:
>
> commit fcfacb9f8374 ("doc: move namespaces.rst from kbuild/ to core-api/")
>
> forgot to update the document kernel-hacking/hacking.rst.
>
> In addition to the fix the path now is a cross-reference to the document.
>
> Signed-off-by: Federico Vaga <federico.vaga@xxxxxxxxxx>
> ---
> Documentation/core-api/symbol-namespaces.rst | 2 ++
> Documentation/kernel-hacking/hacking.rst | 4 ++--
> 2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/core-api/symbol-namespaces.rst b/Documentation/core-api/symbol-namespaces.rst
> index 982ed7b568ac..6791f8a5d726 100644
> --- a/Documentation/core-api/symbol-namespaces.rst
> +++ b/Documentation/core-api/symbol-namespaces.rst
> @@ -1,3 +1,5 @@
> +.. _core-api-namespace:
> +

So I've been wondering for a bit why we don't use section headers as
targets more often rather than adding all these tags. Perhaps it's because
we never enabled that extension? What do you think of this as an
alternative fix? (Probably before committing this I would split into two,
since enabling the extension merits its own patch).

Thanks,

jon