[CMake] FindBoost.cmake updated on the bugtracker

Matthew Woehlke mw_triad at users.sourceforge.net
Wed Apr 2 20:30:21 EDT 2008


Andreas Pakulat wrote:
> Ok, I give in. Do you mind separating the loop into two so its easier to
> see whats going on and we don't have two use-less cache variables?  Then
> I'll apply what you send to kdevplatform and add a new version to the
> cmake bugreport.

Done. I was resisting that because... well, you can look at the patch 
(and see where I had to add redundant code) :-). Although this has the 
elegance of not adding new variables. I decided to wrap the insides of 
both FOREACH's in "IF(NOT Boost_INCLUDE_DIR)" in the theory that an 
empty FOREACH is cheaper than the conditional string manipulation, and 
that it's not worth having said IF twice to avoid an empty FOREACH. Of 
course it would be easy to change this around.

Anyway, patch attached.

-- 
Matthew
 > pinotree uses the large trout on tsdgeos and PutHuhn :)
 > PutHuhn runs
 > tsdgeos lights a fire and eats the trout
(with apologies to Pino Toscano, PutHuhn and Albert Astals Cid, who came 
up with this entirely on their own)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: FindBoost.patch
Type: text/x-patch
Size: 5213 bytes
Desc: not available
URL: <http://www.cmake.org/pipermail/cmake/attachments/20080402/35edfb60/attachment.bin>


More information about the CMake mailing list