<div class="gmail_quote">On Sun, Mar 1, 2009 at 3:55 PM, Alexander Neundorf <span dir="ltr"><<a href="mailto:a.neundorf-work@gmx.net">a.neundorf-work@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Friday 27 February 2009, Philip Lowman wrote:<br>
> On Fri, Feb 27, 2009 at 1:13 PM, Matt Williams <<a href="mailto:lists@milliams.com">lists@milliams.com</a>> wrote:<br>
</div>...<br>
<div class="Ih2E3d">> > There is no indication in the readme.txt which is preferred though I have<br>
> > been<br>
> > assuming the mixed case version. Based on bits and pieces I guess I am<br>
> > right<br>
> > (for example,the "DOXYGEN_FIND_QUIETLY, but it should have been<br>
> > Doxygen_FIND_QUIETLY" comment in FindDoxygen.cmake).<br>
><br>
> I have wondered the same thing for many a moon.<br>
><br>
> The arguments to find_package, i.e. EXACT, QUIET, COMPONENTS, version<br>
> number, etc. cause variables such as Doxygen_FIND_QUIETLY,<br>
> Boost_COMPONENTS, and OpenSceneGraph_FIND_VERSION to be generated by CMake.<br>
> These are based directly off of the name of the find module filename so<br>
> they will generally appear to be mixed case if that is the name of the<br>
> module.<br>
><br>
> Macros like FindPackageHandleStandardArgs seem to make everything upper<br>
> case (i.e DOXYGEN_FOUND, OPENSCENEGRAPH_FOUND, etc.).<br>
><br>
> Not sure what the answer is. It's almost as if input variables are<br>
> encouraged to be the case of the module and output variables are encouraged<br>
> to be uppercase always.<br>
<br>
</div>Currently it is like this:<br>
the variables which are set in a module, are all-upper case. The variables<br>
provided by cmake to the module, are exact-case.<br>
The issue with changing the recommendation to use exact-case for everything is<br>
that most early cmake modules used all-upper-case, and so to stay consistent<br>
we should keep it like this for variables defined by modules.</blockquote><div><br>Can this be added to Modules/readme.txt?<br><br></div></div>-- <br>Philip Lowman<br>