# [PATCH] meson: fix perl dependencies for Documentation 3 messages from 2025-01-11 to 2025-01-14. Participants: Sam James, Patrick Steinhardt. Thread: https://gitlist.dev/t/62787 ## Sam James, 2025-01-11 06:38 Subject: [PATCH] meson: fix perl dependencies for Documentation Message-ID: <79de1bcf31f708960a889066ce3d986e55155ad8.1736577530.git.sam@gentoo.org> URL: https://gitlist.dev/e/79de1bcf31f708960a889066ce3d986e55155ad8.1736577530.git.sam%40gentoo.org ``` We need an explicit `depends: documentation_deps` so that all of our Documentation targets know they require asciidoc.conf. This shows up as parallel build failures with it not yet being available. Other targets look OK already. Signed-off-by: Sam James --- Documentation/technical/meson.build | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build index 21dfb8b5c9..8c93da3566 100644 --- a/Documentation/technical/meson.build +++ b/Documentation/technical/meson.build @@ -42,6 +42,7 @@ api_index = custom_target( '@OUTPUT@', ], env: script_environment, + depends: documentation_deps, input: api_docs, output: 'api-index.txt', ) @@ -60,6 +61,7 @@ foreach article : api_docs + articles command: asciidoc_html_options, input: article, output: fs.stem(article) + '.html', + depends: documentation_deps, install: true, install_dir: get_option('datadir') / 'doc/git-doc/technical', ) -- 2.48.0.rc2 ``` ## Patrick Steinhardt, 2025-01-13 08:26 Subject: Re: [PATCH] meson: fix perl dependencies for Documentation Message-ID: URL: https://gitlist.dev/e/Z4TOTVZbDlxh4Uuy%40pks.im In-Reply-To: <79de1bcf31f708960a889066ce3d986e55155ad8.1736577530.git.sam@gentoo.org> ``` On Sat, Jan 11, 2025 at 06:38:50AM +0000, Sam James wrote: > diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build > index 21dfb8b5c9..8c93da3566 100644 > --- a/Documentation/technical/meson.build > +++ b/Documentation/technical/meson.build > @@ -42,6 +42,7 @@ api_index = custom_target( > '@OUTPUT@', > ], > env: script_environment, > + depends: documentation_deps, > input: api_docs, > output: 'api-index.txt', > ) Wait, this doesn't make a lot of sense to me. "api-index.sh" does not care about "asciidoc.conf" at all, so why is this dependency needed? > @@ -60,6 +61,7 @@ foreach article : api_docs + articles > command: asciidoc_html_options, > input: article, > output: fs.stem(article) + '.html', > + depends: documentation_deps, > install: true, > install_dir: get_option('datadir') / 'doc/git-doc/technical', > ) This one looks as expected, though. Thanks! Patrick ``` ## Sam James, 2025-01-14 14:03 Subject: Re: [PATCH] meson: fix perl dependencies for Documentation Message-ID: <87r055wl06.fsf@gentoo.org> URL: https://gitlist.dev/e/87r055wl06.fsf%40gentoo.org In-Reply-To: ``` Patrick Steinhardt writes: > On Sat, Jan 11, 2025 at 06:38:50AM +0000, Sam James wrote: >> diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build >> index 21dfb8b5c9..8c93da3566 100644 >> --- a/Documentation/technical/meson.build >> +++ b/Documentation/technical/meson.build >> @@ -42,6 +42,7 @@ api_index = custom_target( >> '@OUTPUT@', >> ], >> env: script_environment, >> + depends: documentation_deps, >> input: api_docs, >> output: 'api-index.txt', >> ) > > Wait, this doesn't make a lot of sense to me. "api-index.sh" does not > care about "asciidoc.conf" at all, so why is this dependency needed? You're completely right -- I thought I'd found that api-index.sh *did* need it, but it definitely doesn't upon checking. > >> @@ -60,6 +61,7 @@ foreach article : api_docs + articles >> command: asciidoc_html_options, >> input: article, >> output: fs.stem(article) + '.html', >> + depends: documentation_deps, >> install: true, >> install_dir: get_option('datadir') / 'doc/git-doc/technical', >> ) > > This one looks as expected, though. > > Thanks! I'll respin. Thanks. > > Patrick ```