[geos-devel] [GEOS] #988: Doxygen error check with cmake build
GEOS
geos-trac at osgeo.org
Sat Sep 14 16:13:20 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):
I'm glad you appreciate it :)
"Not documented" warnings are *very* numerous, in fact I ticked that
Doxygen flag fixing less common warnings earlier this year.
The invalid tags comes from updating Doxyfile from 1.2.15 (!) to 1.8.14
with [https://git.osgeo.org/gitea/geos/geos/pulls/81]. There are lags of
varying degrees on different platforms in regard to Doxygen version,
therefore I believe it would be difficult to reach an optimal solution.
(The official [https://geos.osgeo.org/doxygen/] documentation is generated
with v1.8.8 from 2014). I don't think it has a major impact on the result
(although I did notice unexpected result on the official documentation
compared with what I had locally, but that was not necessarily due to
invalid tags).
The reason @strk implemented the check (see: geos/Makefile.am:doxygen-
checked) actively running (successfully!) on Drone, the way he did, I can
only speculate. Above mentioned pull request mirrors its implementation.
I'm aware of the GitHub possibilities, but I didn't want – at this stage –
to push for changes in CI build-files in this PR/commit (the
implementation introduced a new build target {{{make doxygen-checked}}}
(again autotools mirrored), that's why {{{make docs}}} would not do).
Open for suggestions for how to proceed...
--
Ticket URL: <https://trac.osgeo.org/geos/ticket/988#comment:2>
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