On 14/09/2020 07:12, Yonghong Song wrote:
> When building bpf selftests like
> make -C tools/testing/selftests/bpf -j20
> I hit the following errors:
> ...
> GEN /net-next/tools/testing/selftests/bpf/tools/build/bpftool/Documentation/bpftool-gen.8
> <stdin>:75: (WARNING/2) Block quote ends without a blank line; unexpected unindent.
> <stdin>:71: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> <stdin>:85: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> <stdin>:57: (WARNING/2) Block quote ends without a blank line; unexpected unindent.
> <stdin>:66: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> <stdin>:109: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> <stdin>:175: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> <stdin>:273: (WARNING/2) Literal block ends without a blank line; unexpected unindent.
> make[1]: *** [/net-next/tools/testing/selftests/bpf/tools/build/bpftool/Documentation/bpftool-perf.8] Error 12
> make[1]: *** Waiting for unfinished jobs....
> make[1]: *** [/net-next/tools/testing/selftests/bpf/tools/build/bpftool/Documentation/bpftool-iter.8] Error 12
> make[1]: *** [/net-next/tools/testing/selftests/bpf/tools/build/bpftool/Documentation/bpftool-struct_ops.8] Error 12
> ...
>
> I am using:
> -bash-4.4$ rst2man --version
> rst2man (Docutils 0.11 [repository], Python 2.7.5, on linux2)
> -bash-4.4$
>
> Looks like that particular version of rst2man prefers to have a blank line
> after literal blocks. This patch added block lines in related .rst files
> and compilation can then pass.
>
> Cc: Quentin Monnet <quentin@xxxxxxxxxxxxx>
> Fixes: 18841da98100 ("tools: bpftool: Automate generation for "SEE ALSO" sections in man pages")
> Signed-off-by: Yonghong Song <yhs@xxxxxx>
Hi Yonghong, thanks for the fix! I didn't see those warnings on my
setup. For the record my rst2man version is:
rst2man (Docutils 0.16 [release], Python 3.8.2, on linux)
Your patch looks good, but instead of having blank lines at the end of
most files, could you please check if the following works?
------
diff --git a/tools/bpf/bpftool/Documentation/Makefile
b/tools/bpf/bpftool/Documentation/Makefile
index 4c9dd1e45244..01b30ed86eac 100644
--- a/tools/bpf/bpftool/Documentation/Makefile
+++ b/tools/bpf/bpftool/Documentation/Makefile
@@ -32,7 +32,7 @@ RST2MAN_OPTS += --verbose
list_pages = $(sort $(basename $(filter-out $(1),$(MAN8_RST))))
see_also = $(subst " ",, \
- "\n" \
+ "\n\n" \
"SEE ALSO\n" \
"========\n" \
"\t**bpf**\ (2),\n" \
