From: Paul Bauer <paul.bauer.q@gmail.com>
Date: Wed, 13 May 2020 11:46:20 +0000 (+0000)
Subject: Fix check source error parsing
X-Git-Url: http://biod.pnpi.spb.ru/gitweb/?a=commitdiff_plain;h=15cd38a7d407e1491335cd79a62cfd7386de9916;p=alexxy%2Fgromacs.git

Fix check source error parsing

Change-Id: I1a1bb3333fb9a36083b72004e116467432319ae9
---

diff --git a/admin/gitlab-ci/lint.gitlab-ci.yml b/admin/gitlab-ci/lint.gitlab-ci.yml
index b0a7be87b6..d6a24c0f91 100644
--- a/admin/gitlab-ci/lint.gitlab-ci.yml
+++ b/admin/gitlab-ci/lint.gitlab-ci.yml
@@ -154,8 +154,10 @@ check-source:
     - cd $BUILD_DIR
     - cmake --build . --target check-source
     - awk '/warning.*include style.*order/,/You can use.*rst|^$/' docs/doxygen/check-source.log | tee doxygenError.txt || true
+    - awk '/warning:.*includes/,/unnecessarily|^$/' docs/doxygen/check-source.log | tee -a doxygenError.txt || true
     - awk '/Traceback/,/.*rror|^$/' docs/doxygen/doxygen*log docs/doxygen/check-source.log | tee -a doxygenError.txt || true
     - awk '/warning:/,/in doxygen.*|^$/' docs/doxygen/doxygen*log | tee -a doxygenError.txt || true
+    - grep -i "unused cycle suppression" docs/doxygen/check-source.log | tee -a doxygenError.txt || true
     - if [ -s doxygenError.txt ] ; then echo "Found errors while running doxygen"; exit 1; fi
   artifacts:
     name: docs-artifacts-$CI_COMMIT_REF_SLUG
diff --git a/docs/doxygen/cycle-suppressions.txt b/docs/doxygen/cycle-suppressions.txt
index d49152a172..b325f57967 100644
--- a/docs/doxygen/cycle-suppressions.txt
+++ b/docs/doxygen/cycle-suppressions.txt
@@ -2,11 +2,6 @@
 # "moduleA -> moduleB" means that moduleA should not depend on moduleB, and is
 # a problem to be addressed at some point.
 
-# Compat wants assertions and string code from utility, and other string code
-# in utility wants to use string_view from compat. If we fix this, it might be
-# best to implement the compat namespace within the utility module.
-compat -> utility
-
 domdec -> imd
 domdec -> ewald
 domdec -> mdlib
@@ -29,4 +24,4 @@ modularsimulator -> mdrun
 
 # Cycle counters in timing use comrec for the set up, which is in the mdtypes. This introduces
 # cyclic dependencies if the cycle counting is used anywhere in mdtypes.
-timing -> mdtypes
\ No newline at end of file
+timing -> mdtypes
diff --git a/src/gromacs/compat/tests/pointers.cpp b/src/gromacs/compat/tests/pointers.cpp
index 991f9a3172..4131ed2899 100644
--- a/src/gromacs/compat/tests/pointers.cpp
+++ b/src/gromacs/compat/tests/pointers.cpp
@@ -42,8 +42,6 @@
 
 #include "gromacs/compat/pointers.h"
 
-#include "config.h"
-
 #include <memory>
 #include <vector>
 
diff --git a/src/gromacs/gmxpreprocess/tests/readir.cpp b/src/gromacs/gmxpreprocess/tests/readir.cpp
index 80e407451b..71771bcccb 100644
--- a/src/gromacs/gmxpreprocess/tests/readir.cpp
+++ b/src/gromacs/gmxpreprocess/tests/readir.cpp
@@ -47,8 +47,6 @@
 
 #include "gromacs/gmxpreprocess/readir.h"
 
-#include "config.h"
-
 #include <string>
 
 #include <gtest/gtest.h>
diff --git a/src/gromacs/utility/tests/enumerationhelpers.cpp b/src/gromacs/utility/tests/enumerationhelpers.cpp
index 9d27cc96ac..9f7da6257d 100644
--- a/src/gromacs/utility/tests/enumerationhelpers.cpp
+++ b/src/gromacs/utility/tests/enumerationhelpers.cpp
@@ -42,8 +42,6 @@
 
 #include "gromacs/utility/enumerationhelpers.h"
 
-#include "config.h"
-
 #include <iostream>
 
 #include <gtest/gtest.h>
diff --git a/src/programs/mdrun/tests/densityfittingmodule.cpp b/src/programs/mdrun/tests/densityfittingmodule.cpp
index 7149eee08b..ab7cc8798f 100644
--- a/src/programs/mdrun/tests/densityfittingmodule.cpp
+++ b/src/programs/mdrun/tests/densityfittingmodule.cpp
@@ -42,8 +42,6 @@
  */
 #include "gmxpre.h"
 
-#include "config.h"
-
 #include <string>
 
 #include <gmock/gmock.h>