# [PATCH v2] meson: fix perl dependencies for Documentation 3 messages from 2025-01-14 to 2025-01-14. Participants: Sam James, Patrick Steinhardt. Thread: https://gitlist.dev/t/62805 ## Sam James, 2025-01-14 14:03 Subject: [PATCH v2] meson: fix perl dependencies for Documentation Message-ID: <5114dc9a00377826a55f6bab007d2ad1a4de8bc5.1736863427.git.sam@gentoo.org> URL: https://gitlist.dev/e/5114dc9a00377826a55f6bab007d2ad1a4de8bc5.1736863427.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 --- v2: Drop incorrect dep I'd added for api-index.sh as pointed out by Patrick. Documentation/technical/meson.build | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build index 21dfb8b5c9..3a65ee59b3 100644 --- a/Documentation/technical/meson.build +++ b/Documentation/technical/meson.build @@ -60,6 +60,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 ``` ## Patrick Steinhardt, 2025-01-14 14:26 Subject: Re: [PATCH v2] meson: fix perl dependencies for Documentation Message-ID: URL: https://gitlist.dev/e/Z4Z0Fxj87XrqdZl7%40pks.im In-Reply-To: <5114dc9a00377826a55f6bab007d2ad1a4de8bc5.1736863427.git.sam@gentoo.org> ``` On Tue, Jan 14, 2025 at 02:03:47PM +0000, Sam James wrote: > 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 > --- > v2: Drop incorrect dep I'd added for api-index.sh as pointed out by Patrick. > > Documentation/technical/meson.build | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build > index 21dfb8b5c9..3a65ee59b3 100644 > --- a/Documentation/technical/meson.build > +++ b/Documentation/technical/meson.build > @@ -60,6 +60,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', > ) Okay, this looks good to me now, thanks. One last nit on the subject though: as far as I can see the change doesn't have anything to do with Perl, so it is somewhat confusing. How about: meson: fix missing deps for technical articles Thanks! Patrick ``` ## Sam James, 2025-01-14 14:35 Subject: Re: [PATCH v2] meson: fix perl dependencies for Documentation Message-ID: <87ikqhwjjf.fsf@gentoo.org> URL: https://gitlist.dev/e/87ikqhwjjf.fsf%40gentoo.org In-Reply-To: ``` Patrick Steinhardt writes: > On Tue, Jan 14, 2025 at 02:03:47PM +0000, Sam James wrote: >> 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 >> --- >> v2: Drop incorrect dep I'd added for api-index.sh as pointed out by Patrick. >> >> Documentation/technical/meson.build | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Documentation/technical/meson.build b/Documentation/technical/meson.build >> index 21dfb8b5c9..3a65ee59b3 100644 >> --- a/Documentation/technical/meson.build >> +++ b/Documentation/technical/meson.build >> @@ -60,6 +60,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', >> ) > > Okay, this looks good to me now, thanks. One last nit on the subject > though: as far as I can see the change doesn't have anything to do with > Perl, so it is somewhat confusing. How about: > > meson: fix missing deps for technical articles Agreed. Not sure where that came from. I'll send v3 for that then. > > Thanks! > > Patrick ```