[CMake] Does CMake cannot handle directory name with space?

[Philip Lowman]

On Fri, Mar 20, 2009 at 8:03 AM, Dancefire <dancefire at gmail.com> wrote:

> On Fri, Mar 20, 2009 at 10:52 PM, Philip Lowman <philip at yhbt.com> wrote:
>
>> On Fri, Mar 20, 2009 at 2:38 AM, Dancefire <dancefire at gmail.com> wrote:
>>
>>> Hi, Philip,
>>>
>>> I tested your script FindGTK2.cmake on my computer, I got some problems,
>>> but I fixed them. The problem is that I install gtkmm only for current user,
>>> so the path can not be found at
>>> [HKEY_LOCAL_MACHINE\\SOFTWARE\\gtkmm\\2.4;Path], so I add some lines for
>>> [HKEY_CURRENT_USER\\SOFTWARE\\gtkmm\\2.4;Path]. And one more suggestion, I
>>> don't know how, but if it is possible to use the environment variable
>>> "GTKMM_BASEPATH", which contain the gtkmm installation base path and setup
>>> by installer, then it will more easily locate the gtkmm.
>>
>>
>> Thanks for the patches.  I'll commit the module to CVS sometime this
>> weekend with your improvements.  Do you think it would be better to put the
>> HKEY_CURRENT_USER before HKEY_CURRENT_MACHINE to allow a user install to
>> override the system install?
>>
>
> Yeah, it would be better let HKCU override HKLM. If there is a difference,
> then the current user most likely intends to use its own installation.
>
>
>>
>> It is possible to use environment variables to find stuff.  The syntax for
>> this is $ENV{GTKMM_BASEPATH}.  I'll put this before all the registry keys.
>>
>>
>
> That's great, thank you.
>

If you could give the code I just checked into CVS a test that would be
great.  I added version support, all of the aforementioned stuff, and
cleaned things up a bit.  I also made it so calling the module with no
COMPONENTS searches only for GTK since I believe this was suggested earlier
(and not doing so would break CMake's dashboard).

http://www.cmake.org/cgi-bin/viewcvs.cgi/Modules/FindGTK2.cmake?root=CMake&view=markup

-- 
Philip Lowman
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.cmake.org/pipermail/cmake/attachments/20090322/1f4b6906/attachment.htm>