git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 3/3] ci: don't compile whole project when testing docs with Meson

From
Karthik Nayak <karthik.188@gmail.com>
Date
Sep 16, 2025, 09:16 UTC
Message-ID
<CAOLa=ZSXXk9iwnU6a9BR-jfhs0XU8MXQqa8edpOajfORV2uY5w@mail.gmail.com>
In-Reply-To
<20250911-b4-pks-meson-docs-target-v1-3-a92c666ecef9@pks.im>
Patrick Steinhardt <ps@pks.im> writes:
Show 38 quoted lines
> Our "documentation" CI jobs, unsurprisingly, performs a couple of tests
> on our documentation. The job knows to not only test the documentation
> generated by our Makefile, but also by Meson.
>
> In the latter case with Meson we end up building the whole project,
> including all of the binaries. This is of course quite excessive and a
> waste of compute cycles, as we don't care about these binaries at all.
>
> Fix this by using the new "docs" target that we introduced in the
> preceding commit.
>
> Reported-by: SZEDER Gábor <szeder.dev@gmail.com>
> Signed-off-by: Patrick Steinhardt <ps@pks.im>
> ---
>  ci/test-documentation.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/ci/test-documentation.sh b/ci/test-documentation.sh
> index 49f87f50fd..5e4fd8fbd7 100755
> --- a/ci/test-documentation.sh
> +++ b/ci/test-documentation.sh
> @@ -48,13 +48,13 @@ check_unignored_build_artifacts
>
>  # Build docs with Meson and AsciiDoc
>  meson setup build-asciidoc -Ddocs=html,man -Ddocs_backend=asciidoc
> -meson compile -C build-asciidoc
> +meson compile -C build-asciidoc docs
>  check_docs build-asciidoc AsciiDoc
>  rm -rf build-asciidoc
>
>  # Build docs with Meson and AsciiDoctor
>  meson setup build-asciidoctor -Ddocs=html,man -Ddocs_backend=asciidoctor
> -meson compile -C build-asciidoctor
> +meson compile -C build-asciidoctor docs
>  check_docs build-asciidoctor Asciidoctor
>  rm -rf build-asciidoctor
>
>

This makes sense. I really like how a lot of our CI config is forge agnostic. I opened the patch expecting to review GitHub/GitLab changes. Glad I didn't have to.

> --
> 2.51.0.450.g87641ccf93.dirty
Previous: Patrick SteinhardtNext: Junio C Hamano
Message 8 of 14 in “ci: don't compile whole project when testing docs with Meson”
  1. 0/3 ci: don't compile whole project when testing docs with MesonPatrick Steinhardt, Sep 11, 2025
  2. 1/3 meson: introduce a "docs" alias to compile documentation onlyPatrick Steinhardt, Sep 11, 2025
  3. Karthik NayakSep 16, 2025
  4. Patrick SteinhardtSep 16, 2025
  5. Karthik NayakSep 16, 2025
  6. 2/3 meson: print docs backend as part of the summaryPatrick Steinhardt, Sep 11, 2025
  7. 3/3 ci: don't compile whole project when testing docs with MesonPatrick Steinhardt, Sep 11, 2025
  8. Karthik NayakSep 16, 2025
  9. Junio C HamanoSep 11, 2025
  10. Patrick SteinhardtSep 15, 2025
  11. Junio C HamanoSep 15, 2025
  12. Patrick SteinhardtSep 16, 2025
  13. Junio C HamanoSep 16, 2025
  14. Karthik NayakSep 16, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.