[geos-devel] [GEOS] #988: Doxygen error check with cmake build

GEOS geos-trac at osgeo.org
Mon Sep 16 05:01:15 PDT 2019


#988: Doxygen error check with cmake build
-----------------------------------+---------------------------
 Reporter:  nila                   |       Owner:  geos-devel@…
     Type:  enhancement            |      Status:  new
 Priority:  major                  |   Milestone:
Component:  Build/Install (cmake)  |     Version:  master
 Severity:  Unassigned             |  Resolution:
 Keywords:                         |
-----------------------------------+---------------------------

Comment (by nila):

 As I suspected @strk, but didn't want to put words in your mouth. And I
 agree. Better to keep all warnings in logfile, what's hidden is forgotten.
 I made a small change in the error check cmake-script, which states in
 build log how many warnings had been issued:
 e.g.

 {{{-- Doxygen issued 1702 warning(s), see
 /home/travis/build/libgeos/geos/_build/doc/doxygen.log}}}

 regardless if the build succeeds or not.

 I've tested reverting the Doxygen.in file to version 1.8.8, and setting
 WARN_IF_UNDOCUMENTED to NO. Building with 1.8.15 didn't issue any
 warnings, so this could be a way to go, but in the future obsolete tags
 might appear as well, so... For now, I think we should decide what
 warnings we can allow to be ignored.

 @dbaston I opened a PR on github as well
 [https://github.com/libgeos/geos/pull/224], with intentional doxygen
 errors and the {{{doxygen-checked}}} target activated on travis, and the
 build failed as expected:

 {{{
 Built target docs
 Scanning dependencies of target doxygen-checked
 -- Doxygen issued 1702 warning(s), see
 /home/travis/build/libgeos/geos/_build/doc/doxygen.log
 CMake Error at check_doxygen_errors.cmake:42 (message):
   /home/travis/build/libgeos/geos/include/geos/noding/SegmentNode.h:70:
   warning: argument 'THISISCOMPLETELYWRONGTOO' of command @param is not
 found
   in the argument list of geos::noding::SegmentNode::SegmentNode(const
   NodedSegmentString &ss, const geom::Coordinate &nCoord, size_t
   nSegmentIndex, int nSegmentOctant)

   /home/travis/build/libgeos/geos/include/geos/planargraph/Edge.h:88:
   warning: argument 'THISISCOMPLETELYWRONG' of command @param is not found
 in
   the argument list of geos::planargraph::Edge::Edge(DirectedEdge *de0,
   DirectedEdge *de1)

     parameter 'de0'
     parameter 'de1'


 doc/CMakeFiles/doxygen-checked.dir/build.make:57: recipe for target
 'doc/CMakeFiles/doxygen-checked' failed
 make[3]: *** [doc/CMakeFiles/doxygen-checked] Error 1
 CMakeFiles/Makefile2:1888: recipe for target 'doc/CMakeFiles/doxygen-
 checked.dir/all' failed
 make[2]: *** [doc/CMakeFiles/doxygen-checked.dir/all] Error 2
 CMakeFiles/Makefile2:1895: recipe for target 'doc/CMakeFiles/doxygen-
 checked.dir/rule' failed
 make[1]: *** [doc/CMakeFiles/doxygen-checked.dir/rule] Error 2
 Makefile:795: recipe for target 'doxygen-checked' failed
 make: *** [doxygen-checked] Error 2
 The command "./tools/ci/script.sh" exited with 2.
 }}}

 Now, it should be tested on Windows too (I've tested on macOS). @dbaston,
 could you please help out with this?

-- 
Ticket URL: <https://trac.osgeo.org/geos/ticket/988#comment:4>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).


More information about the geos-devel mailing list