MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010199CMakeModulespublic2010-01-29 03:452010-11-05 12:09
ReporterMarcel Loose 
Assigned ToMarcel Loose 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake-2-8 
Target VersionCMake 2.8.3Fixed in VersionCMake 2.8.3 
Summary0010199: FindSubversion: mismatch between code and documentation
DescriptionThere are two mismatches between the code and the documentation in FindSubversion:

1) The variable <var-prefix>_WC_ROOT is documented, but is not set.

2) The documentation doesn't mention the existence of a second macro
Subversion_WC_LOG, and erroneously mentions that Subversion_WC_INFO sets
the variable <var-prefix>_WC_LAST_CHANGED_LOG. This variable is called
<var-prefix>_LAST_CHANGED_LOG and is set by Subversion_WC_LOG.

I've added a patch that solves these issues.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch FindSubversion.cmake.wc_info.patch (2,473) 2010-01-29 03:45
https://public.kitware.com/Bug/file/2820/FindSubversion.cmake.wc_info.patch
Issue History
Date ModifiedUsernameFieldChange
2010-01-29 03:45Marcel LooseNew Issue
2010-01-29 03:45Marcel LooseFile Added: FindSubversion.cmake.wc_info.patch
2010-08-31 17:00David ColeNote Added: 0022041
2010-09-13 03:58Marcel LooseAssigned To => Marcel Loose
2010-09-13 03:58Marcel LooseStatusnew => assigned
2010-09-13 07:43Marcel LooseNote Added: 0022207
2010-09-13 07:44Marcel LooseStatusassigned => resolved
2010-09-13 08:19David ColeFixed in Version => CMake 2.8.3
2010-09-13 08:19David ColeTarget Version => CMake 2.8.3
2010-11-05 12:09Marcel LooseStatusresolved => closed
2010-11-05 12:09Marcel LooseResolutionopen => fixed

Notes
(0022041)
David Cole   
2010-08-31 17:00   
We will not be addressing this for the upcoming CMake 2.8.3 release.

(Unless the existing module maintainer responds, or a new one comes forward to take up the mantle...)
(0022207)
Marcel Loose   
2010-09-13 07:43   
Applied the patch.