From 6b9a9807953649a702f883dd263366079fcc02b0 Mon Sep 17 00:00:00 2001 From: Tomasz Zawadzki Date: Wed, 24 Nov 2021 10:20:33 +0100 Subject: [PATCH] autobuild: skip doxygen 1.8.20 false positives Doxygen 1.8.20 contains false positives that are fixed in later version. Right now if that version of doxygen is used in tests, any error is ignored. Please see: (63330f6)autobuild: Ignore warnings comming from doxygen 1.8.20 The false positive is realted to: https://github.com/doxygen/doxygen/issues/7948 This patch ignores only warnings releated to "\ilinebr" processing in doxygen, reporting any other warnings. Signed-off-by: Tomasz Zawadzki Change-Id: I19595f35a2e6f66b9917133a19cf45394a240dbd Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10388 Tested-by: SPDK CI Jenkins Community-CI: Broadcom CI Reviewed-by: Monica Kenguva Reviewed-by: Ben Walker Reviewed-by: Michal Berger Reviewed-by: Jim Harris --- autobuild.sh | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/autobuild.sh b/autobuild.sh index bf91951f82..8b34f3920f 100755 --- a/autobuild.sh +++ b/autobuild.sh @@ -273,10 +273,18 @@ function build_doc() { $MAKE -C "$rootdir"/doc --no-print-directory $MAKEFLAGS &> "$out"/doxygen.log if [ -s "$out"/doxygen.log ]; then - cat "$out"/doxygen.log - echo "Doxygen errors found!" - eq "$doxygenv" 1.8.20 || exit 1 - echo "Doxygen $doxygenv detected, all warnings are potentially false positives, continuing the test" + if [[ "$doxygenv" != "1.8.20" ]]; then + cat "$out"/doxygen.log + echo "Doxygen errors found!" + exit 1 + fi + # Doxygen 1.8.20 produces false positives, see: + # https://github.com/doxygen/doxygen/issues/7948 + if grep -v "\ilinebr" "$out"/doxygen.log; then + echo "Doxygen errors found!" + exit 1 + fi + echo "Doxygen $doxygenv detected. No warnings except false positives, continuing the test" fi if hash pdflatex 2> /dev/null; then $MAKE -C "$rootdir"/doc/output/latex --no-print-directory $MAKEFLAGS &>> "$out"/doxygen.log