Anastasia Klimchuk has uploaded this change for review. ( 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 --- M doc/meson.build M test_build.sh 2 files changed, 9 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/65/81665/1
diff --git a/doc/meson.build b/doc/meson.build index 5bc57b9..a2dad0e 100644 --- a/doc/meson.build +++ b/doc/meson.build @@ -34,4 +34,12 @@ ) endif
+else # sphinx-build not found + if get_option('man-pages').enabled() + error('Cannot build man-pages because sphinx-build is missing.') + endif + + if get_option('documentation').enabled() + error('Cannot build documentation because sphinx-build is missing.') + endif endif diff --git a/test_build.sh b/test_build.sh index 9b490dc..081738b 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}