Anastasia Klimchuk submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Martin L Roth: Looks good to me, approved Anton Samsonov: Looks good to me, but someone else must approve
meson: Update CI script to enforce building man pages and docs

`test_build.sh` is used by Jenkins, therefore it should build
everything. Docker container for Jenkins is expected to have all
the dependencies installed, and if some of them are missing, script
should fail.

Recently we had a situation when docker image was missing sphinx
and flashrom Jenkins was silently skipping building man-pages and
documentation. This patch prevents this happening again, because
building man pages and docs will be enforced.

Change-Id: Ib89abddad27d1168cf0a621cf4bdb9f541266165
Signed-off-by: Anastasia Klimchuk <aklm@flashrom.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/81665
Reviewed-by: Martin L Roth <gaumless@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anton Samsonov <avscomputing@gmail.com>
---
M test_build.sh
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test_build.sh b/test_build.sh
index 9b490dc..ee86496 100755
--- a/test_build.sh
+++ b/test_build.sh
@@ -51,7 +51,7 @@

build_meson () {
build_dir=out
- meson_opts="-Dtests=enabled"
+ meson_opts="-Dtests=enabled -Dman-pages=enabled -Ddocumentation=enabled"
ninja_opts="-j $(nproc)"

rm -rf ${build_dir}

To view, visit change 81665. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: Ib89abddad27d1168cf0a621cf4bdb9f541266165
Gerrit-Change-Number: 81665
Gerrit-PatchSet: 4
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Anton Samsonov <avscomputing@gmail.com>
Gerrit-Reviewer: Anton Samsonov <devel@zxlab.ru>
Gerrit-Reviewer: Martin L Roth <gaumless@gmail.com>
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged