Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/81665?usp=email )
Change subject: meson: Update CI script to enforce building man pages and docs ......................................................................
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(-)
Approvals: build bot (Jenkins): Verified Martin L Roth: Looks good to me, approved Anton Samsonov: Looks good to me, but someone else must approve
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}