MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0008054CMakeModulespublic2008-11-07 04:572009-02-23 22:44
ReporterSpringer 
Assigned ToPhilip Lowman 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake-2-6 
Target VersionFixed in VersionCMake-2-6 
Summary0008054: FindDoxygen.cmake doesn't set all variables correctly
DescriptionThe module FindDoxygen.cmake sets DOXYGEN_FOUND to "YES" if doxygen was found and at the end of the module this variable (among others) is marked as advanced.

However, another variable DOXYGEN_DOT_FOUND, which is also marked as advanced there, is never set. Expected behaviour: Similarly set DOXYGEN_DOT_FOUND to "YES" if dot was found, otherwise to "NO". This would allow this variable to be directly moved via configure_file() to the doxygen configuration file. Doxygen expects either a "YES" or "NO" there.

For consistency, I'd suggest to also set DOXYGEN_FOUND to "NO" if doxygen has not been found. Currently the value of that variable is not changed in that case, so it's probably unset.

Additionally, I think these two variables should be documented. The current documentation doesn't mention them at all.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-11-07 04:57SpringerNew Issue
2009-01-10 08:44Alex NeundorfCategoryCMake => Modules
2009-01-15 13:00Bill HoffmanStatusnew => assigned
2009-01-15 13:00Bill HoffmanAssigned To => Philip Lowman
2009-01-19 22:28Philip LowmanNote Added: 0014640
2009-01-19 22:28Philip LowmanStatusassigned => resolved
2009-01-19 22:28Philip LowmanResolutionopen => fixed
2009-02-23 22:44Philip LowmanNote Added: 0015308
2009-02-23 22:44Philip LowmanStatusresolved => closed
2009-02-23 22:44Philip LowmanFixed in Version => CMake-2-6

Notes
(0014640)
Philip Lowman   
2009-01-19 22:28   
Springer,
I fixed all of the issues you mentioned and more. Thanks for reporting this.
You can download the latest version here:
http://public.kitware.com/cgi-bin/viewcvs.cgi/Modules/?root=CMake [^]

DOXYGEN_DOT_FOUND now exists, errant
mark_as_advanced variables removed, documentation cleaned up and OSX stuff
isolated to it's own section, support added for DOXYGEN_SKIP_DOT, support
added to call FindPackageHandleStandardArgs to avoid output on every CMake
run.

lowman@locke:~/cvs/CMake_write/Modules$ cvs commit FindDoxygen.cmake
Committer: Philip Lowman <philip@yhbt.com>
/cvsroot/CMake/CMake/Modules/FindDoxygen.cmake,v <-- FindDoxygen.cmake
new revision: 1.16; previous revision: 1.15
(0015308)
Philip Lowman   
2009-02-23 22:44   
Fix available in 2.6.3