Ok no worries, keep with the minimal dependency approach. Things will be much cleaner in the end.<div><br></div><div><br></div><div>Regards</div><div>  Andrew</div><div><br><br><div class="gmail_quote">On Thu, Mar 10, 2011 at 1:47 AM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com">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;">Hi Andrew,<br>
<div class="im"><br>
On Tue, Mar 8, 2011 at 11:13 PM, Andrew Maclean<br>
<<a href="mailto:andrew.amaclean@gmail.com">andrew.amaclean@gmail.com</a>> wrote:<br>
> Hi Marcus,<br>
>    Firstly congratulations to you all on a brilliant first effort.<br>
> One minor change, could I suggest moving:<br>
> [Filters/Sources]<br>
><br>
> Graphics/vtkParametricFunctionSource.cxx:<br>
> Graphics/vtkParametricFunctionSource.h:<br>
><br>
><br>
> To:<br>
> [Core/ComputationalGeometry]<br>
> Logically vtkParametricFunctionSource belongs with the rest of the<br>
> vtkParametric classes.<br>
<br>
</div>This is one case where it would be nice to keep it where it is. The<br>
Core/ComputationalGeometry module should only depend on Core/Core<br>
right now, but if we made this move then it would also depend on<br>
Core/ExecutionModel. Filters/Sources would of course link to<br>
Core/ExecutionModel, and to Core/ComputationalGeometry if necessary.<br>
<br>
That said I have not gotten this far with the build system, but I<br>
think it would be nice to keep the minimal dependency set for<br>
ComputationalGeometry if this is feasible.<br>
<font color="#888888"><br>
Marcus<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>___________________________________________<br>Andrew J. P. Maclean<br>Australian Centre for Field Robotics (ACFR)<br>The Rose Street Building J04<br>The University of Sydney  2006  NSW<br>
AUSTRALIA<br>Ph: +61 2 9351 3283<br>Fax: +61 2 9351 7474<br>URL: <a href="http://www.acfr.usyd.edu.au/" target="_blank">http://www.acfr.usyd.edu.au/</a><br>___________________________________________<br>
</div>