From 273be0b9e61246e932a776b35de29c6bf6d85b41 Mon Sep 17 00:00:00 2001 From: Dimitrios Apostolou Date: Sat, 4 Jul 2020 01:32:40 +0200 Subject: [PATCH] Skip LGTM analysis for the bootstrap library and tools This is because the same code is built for the bootstrap library with non-standard options like disabled threads, which causes false alerts. So we build the bootstrap tools in the "configure" step. Additionally we update the configure flags to only the ones that matter for the CMake build, after the latest configure script updates. Change-Id: Id4f90db2a16904f6665f3994f23a9f2a974aefc6 Reviewed-by: Volker Hilsheimer --- .lgtm.yml | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/.lgtm.yml b/.lgtm.yml index 3a399d4862..2619c647d4 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -16,8 +16,18 @@ extraction: \ | tar -xzf - --strip-components=1 -C $HOME/cmake-3.17 - export PATH=$HOME/cmake-3.17/bin:$PATH + # Pre-analysis step. configure: - command: "./configure -cmake -opensource -confirm-license -debug -no-optimize-debug \ - \ -nomake tests -nomake examples -no-harfbuzz -no-iconv -system-pcre -system-zlib \ - \ -system-freetype -system-libpng -system-libjpeg -system-sqlite -- -DFEATURE_system_sqlite=ON \ - \ -DQT_NO_MAKE_TESTS=ON -DQT_NO_MAKE_EXAMPLES=ON" + command: + - "./configure -cmake -opensource -confirm-license -debug -no-pch \ + \ -nomake tests -nomake examples -no-harfbuzz \ + \ -system-pcre -system-zlib -system-freetype -system-libpng \ + \ -system-libjpeg -system-sqlite" + # We skip analyzing the bootstrap library, since the same code is + # built with non-standard flags and false-positives are flagged. + - ninja bootstrap_tools + + # Actual analysis. + index: + build_command: + - ninja