<div dir="ltr">+1<div><br></div><div style>If we bump, what about requiring CMake 2.8.11 ? You could then use INTERFACE_INCLUDE_DIRECTORIES [1], or any of the INTERFACE_* property</div><div style><br></div><div style>[1] <a href="http://cmake.org/cmake/help/v2.8.11/cmake.html#prop_tgt:INTERFACE_INCLUDE_DIRECTORIES">http://cmake.org/cmake/help/v2.8.11/cmake.html#prop_tgt:INTERFACE_INCLUDE_DIRECTORIES</a></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 30, 2013 at 3:34 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jul 30, 2013 at 3:06 PM, Marcus D. Hanwell<br>
<<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
> Hi,<br>
><br>
> I propose that we bump our CMake minimum version to from 2.8.5 to<br>
> 2.8.8 (going from a version released in July 2011 to one released in<br>
> April 2012). My main motivation is so that we can add private<br>
> dependencies to modules using the public/private interface library<br>
> API, and make use of per-target include directories. There are several<br>
> other minor feature improvements and additions, and with 6.0 just<br>
> released we have approximately half a year until distributions will<br>
> begin packaging this version.<br>
><br>
> Private dependencies would let us clearly define what is an interface<br>
> library, and what is limited to implementation and does not need to be<br>
> added to link lines for dependent libraries (when building shared at<br>
> least). This would also allow us to reduce the number of include<br>
> directories to those in the current private set and the accumulated<br>
> public set and should improve compilation speed.<br>
><br>
> I think most of our dashboards already have 2.8.8 as other projects<br>
> already require this or later, and it would allow us to simplify some<br>
> of our CMake code that was condiitionally using some of these new<br>
> features.<br>
><br>
</div>To add, the oldest we currently test/verify on the dashboards is<br>
2.8.7, and several of those machines would need to be updated to at<br>
least 2.8.8.<br>
<div class="HOEnZb"><div class="h5"><br>
Marcus<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>+1 919 869 8849<br>
</div>