[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y/ZlDnioTn+hj03/@google.com>
Date: Wed, 22 Feb 2023 11:55:10 -0700
From: Ross Zwisler <zwisler@...gle.com>
To: Bagas Sanjaya <bagasdotme@...il.com>
Cc: Linux BPF <bpf@...r.kernel.org>,
Linux Documentation <linux-doc@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Alexei Starovoitov <ast@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
Andrii Nakryiko <andrii@...nel.org>,
Martin KaFai Lau <martin.lau@...ux.dev>,
Song Liu <song@...nel.org>, Yonghong Song <yhs@...com>,
John Fastabend <john.fastabend@...il.com>,
KP Singh <kpsingh@...nel.org>,
Stanislav Fomichev <sdf@...gle.com>,
Hao Luo <haoluo@...gle.com>, Jiri Olsa <jolsa@...nel.org>,
Jonathan Corbet <corbet@....net>,
Alan Maguire <alan.maguire@...cle.com>
Subject: Re: [PATCH v2] Documentation: bpf: Fix link to BTF doc
On Wed, Feb 22, 2023 at 03:35:30PM +0700, Bagas Sanjaya wrote:
> Ross reported broken link to BTF documentation
> (Documentation/bpf/btf.rst) in Documentation/bpf/bpf_devel_QA.rst. The
> link in question is written using external link syntax, with the target
> refers to BTF doc in reST source (btf.rst), which doesn't exist in
> resulting HTML output.
>
> Fix the link by replacing external link syntax with simply writing out
> the target doc, which the link will be generated to the correct HTML doc
> target.
>
> Link: https://lore.kernel.org/linux-doc/Y++09LKx25dtR4Ow@google.com/
> Fixes: 6736aa793c2b5f ("selftests/bpf: Add general instructions for test execution")
> Reported-by: Ross Zwisler <zwisler@...gle.com>
> Signed-off-by: Bagas Sanjaya <bagasdotme@...il.com>
> ---
> Changes since v1 [1]:
>
> * Reword patch description (I don't see external link semantics on
> Sphinx documentation [2] when I submit v1).
> * Drop the corresponding orphan target definition.
> * Rebase on top of current bpf tree.
>
> Ross, do you want to give a Reviewed-by or Acked-by?
Sure, thanks for the fix:
Acked-by: Ross Zwisler <zwisler@...gle.com>
Powered by blists - more mailing lists