#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 [https://github.com/libgeos/geos/pull/224#discussion_r324756010
 suggested] by @dbaston I made the doxygen error check a cmake test
 (instead of a new build target). I have now cleaned up commit history and
 removed intentional test-bugs. Build test on travis works fine (although
 at the moment doxygen errors cause failure, something
 [https://git.osgeo.org/gitea/geos/geos/pulls/97 PR#97] probably will
 solve).

 As the test is only executed with doxygen installed, it will not affect
 present Windows or macOS CI builds. It might not be too great a risk to go
 for this before further testing and deal with eventual problems as they
 present themselves?!

 As I see it, the PR is now ready for merge.

-- 
Ticket URL: <https://trac.osgeo.org/geos/ticket/988#comment:5>
GEOS <http://trac.osgeo.org/geos>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite 
(JTS).
_______________________________________________
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

Reply via email to