Re: [PATCH] perf tools: Fix man page build wrt perf-arm-coresight.txt

From: Leo Yan
Date: Mon Oct 17 2022 - 05:57:56 EST


On Mon, Oct 17, 2022 at 12:35:49PM +0300, Adrian Hunter wrote:
> perf build assumes documentation files starting with "perf-" are man
> pages but perf-arm-coresight.txt is not a man page:
>
> asciidoc: ERROR: perf-arm-coresight.txt: line 2: malformed manpage title
> asciidoc: ERROR: perf-arm-coresight.txt: line 3: name section expected
> asciidoc: FAILED: perf-arm-coresight.txt: line 3: section title expected
> make[3]: *** [Makefile:266: perf-arm-coresight.xml] Error 1
> make[3]: *** Waiting for unfinished jobs....
> make[2]: *** [Makefile.perf:895: man] Error 2
>
> Fix by renaming it.
>
> Reported-by: Sven Schnelle <svens@xxxxxxxxxxxxx>
> Reported-by: Christian Borntraeger <borntraeger@xxxxxxxxxxxxx>
> Fixes: dc2e0fb00bb2 ("perf test coresight: Add relevant documentation about ARM64 CoreSight testing")
> Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>

Thanks for fixing, Adrian. The change LGTM:

Reviewed-by: Leo Yan <leo.yan@xxxxxxxxxx>

> ---
> .../Documentation/{perf-arm-coresight.txt => arm-coresight.txt} | 0
> 1 file changed, 0 insertions(+), 0 deletions(-)
> rename tools/perf/Documentation/{perf-arm-coresight.txt => arm-coresight.txt} (100%)
>
> diff --git a/tools/perf/Documentation/perf-arm-coresight.txt b/tools/perf/Documentation/arm-coresight.txt
> similarity index 100%
> rename from tools/perf/Documentation/perf-arm-coresight.txt
> rename to tools/perf/Documentation/arm-coresight.txt
> --
> 2.25.1
>