[PATCH] trace-cmd documentation: Remove check-manpages.sh for trace-cmd in meson build

[PATCH] trace-cmd documentation: Remove check-manpages.sh for trace-cmd in meson build

[Steven Rostedt]

From: "Steven Rostedt (Google)" <rostedt@goodmis.org>

The creation of man pages for trace-cmd ran the check-manpages.sh script,
which is only used for the libtracecmd library, not the tools. This caused
the build to spit out warning of grep not finding libtracecmd.txt.

Fixes: 736df0648d16b ("trace-cmd: Add initial support for meson")
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
---
 Documentation/trace-cmd/meson.build | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/Documentation/trace-cmd/meson.build b/Documentation/trace-cmd/meson.build
index d0cc3076d985..875d72832a0b 100644
--- a/Documentation/trace-cmd/meson.build
+++ b/Documentation/trace-cmd/meson.build
@@ -104,13 +104,6 @@ if get_option('docbook-suppress-sp')
     xmlto_extra += ['-m ',  confdir + 'manpage-suppress-sp.xsl']
 endif
 
-check_doc = custom_target(
-    'check-doc',
-    output: 'dummy',
-    command : [
-        top_source_dir + 'check-manpages.sh',
-        meson.current_source_dir()])
-
 gen = generator(
     asciidoc,
     output: '@BASENAME@.xml',
@@ -131,7 +124,6 @@ foreach txt, section : sources
         txt.underscorify() + '_man',
         input: xml,
         output: '@BASENAME@.' + section,
-        depends: check_doc,
         command: [
             xmlto,
             '-m', manpage_xsl,
@@ -145,7 +137,6 @@ foreach txt, section : sources
         txt.underscorify() + '_html',
         input: txt,
         output: '@BASENAME@.html',
-        depends: check_doc,
         command: [
             asciidoc,
             '-b', asciidoc_html,
-- 
2.43.0

Re: [PATCH] trace-cmd documentation: Remove check-manpages.sh for trace-cmd in meson build

[Daniel Wagner]

On Thu, Jul 25, 2024 at 11:26:20AM GMT, Steven Rostedt wrote:
> From: "Steven Rostedt (Google)" <rostedt@goodmis.org>
> 
> The creation of man pages for trace-cmd ran the check-manpages.sh script,
> which is only used for the libtracecmd library, not the tools. This caused
> the build to spit out warning of grep not finding libtracecmd.txt.

Ah, okay, didn't noticed that back then.

> Fixes: 736df0648d16b ("trace-cmd: Add initial support for meson")
> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>

Anyway, looks good.

Reviewed-by: Daniel Wagner <dwagner@suse.de>