<p>Mathieu, </p>
<p>It's a nice idea, but given that Boost could make any one of its libraries dependent on "system" at any time they want in the future (and for all I know there are others already dependent on it, besides wave and filesystem), I'd rather the user be responsible for adding "system" manually. It's not too hard to figure this out from the link errors. </p>
<p><blockquote type="cite">On Nov 4, 2009 9:40 AM, "Mathieu Malaterre" <<a href="mailto:mathieu.malaterre@gmail.com">mathieu.malaterre@gmail.com</a>> wrote:<br><br>Philip,<br>
<br>
I guess I was not very clear in my previous email. But here is my<br>
proposed change:<br>
<br>
@@ -322,8 +322,14 @@<br>
if(Boost_VERSION AND Boost_FIND_COMPONENTS)<br>
math(EXPR _boost_maj "${Boost_VERSION} / 100000")<br>
math(EXPR _boost_min "${Boost_VERSION} / 100 % 1000")<br>
- if(${_boost_maj}.${_boost_min} VERSION_LESS 1.35)<br>
- list(REMOVE_ITEM Boost_FIND_COMPONENTS system)<br>
+ if(${_boost_maj}.${_boost_min} VERSION_GREATER 1.34)<br>
+ # when use asked for filesystem or wave, automatically add<br>
+ # system to the list of components<br>
+ list(FIND Boost_FIND_COMPONENTS filesystem v1)<br>
+ list(FIND Boost_FIND_COMPONENTS wave v2)<br>
+ if(v1 OR v2)<br>
+ list(APPEND Boost_FIND_COMPONENTS system)<br>
+ endif(v1 OR v2)<br>
endif()<br>
endif()<br>
<br>
<br>
Therefore calling<br>
<p><font color="#500050">
FIND_PACKAGE ( Boost COMPONENTS filesystem REQUIRED)
</font></p>will work on boost 1.34 and boost > 1.34<br>
<br>
Comments ?<br>
<br>
On Tue, Nov 3, 2009 at 3:20 PM, Mathieu Malaterre<br>
<p><font color="#500050"><<a href="mailto:mathieu.malaterre@gmail.com">mathieu.malaterre@gmail.com</a>> wrote:
> The way I see it to address is that 'system' is an internal d...</font></p>--<br>
<font color="#888888">Mathieu<br>
</font></blockquote></p>