<div>Some of my thoughts:</div><div><br></div><div>1). I agreed that we haven't take full advantage of modular VTK in ITK yet.</div><div> VtkGlue module itself should only depends on several VTK modules instead of entire VTK_LIBRAIRES.</div>
<div><br></div><div>2) When building ITK, ITK_LIBRAIRES should include all depending libraries turned on by enabled modules.</div><div> Users then should be able do </div><div><div><i> Find( ITK Required ITKCommon ITKVtkGlue) </i></div>
<div> in their itk application code without specifying "Find(VTK .....)" in their own code, unless their code depends on</div><div> some other vtk modules which are not specified in VtkGlue's dependency list.</div>
<div> </div><div><br></div><div> <br><br><div class="gmail_quote">On Tue, Jun 26, 2012 at 1:16 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The more I think about it, it may not be a bug<br>
<br>
VTK modular lets us just link against certain modules.<br>
<br>
for example, FIND(VTK REQUIRED vtkOpenGLRendering)<br>
<br>
ITK and your App may have different module requirements.<br>
<br>
We are not taking advantage of this yet.<br>
<span class="HOEnZb"><font color="#888888"><br>
Bill<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Jun 26, 2012 at 1:10 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> After VTK modular, the USE_VTK_FILE does a lot more. I think for the<br>
> time being, you will have to<br>
> FIND_PACKAGE(VTK)<br>
> INCLUDE(${USE_VTK_FILE})<br>
><br>
> the ITK configuration is not smart enough yet.<br>
><br>
> It's definitely a bug.<br>
><br>
> On Tue, Jun 26, 2012 at 1:07 PM, David Doria <<a href="mailto:daviddoria@gmail.com">daviddoria@gmail.com</a>> wrote:<br>
>> On Tue, Jun 26, 2012 at 12:58 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> I think you still need<br>
>>> ${VTK_LIBRARIES}<br>
>><br>
>><br>
>> I actually don't need ${VTK_LIBRARIES},<br>
>><br>
>> TARGET_LINK_LIBRARIES(TestITK ${ITK_LIBRARIES})<br>
>><br>
>> is sufficient. However, what I do need is:<br>
>><br>
>> FIND_PACKAGE(VTK)<br>
>> INCLUDE(${USE_VTK_FILE})<br>
>><br>
>> Which I agree is not right :)<br>
>><br>
>> David<br>
><br>
><br>
><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
<br>
<br>
<br>
--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><br></div><div><br></div>---------------------------------------------<br><b>Xiaoxiao Liu</b>, Ph.D.<div><div><div><font face="arial, sans-serif"><span style="border-collapse:collapse">R & D Engineer</span></font></div>
<div><div><a href="http://www.kitware.com/" target="_blank">Kitware Inc</a>.</div></div><div><span style="border-collapse:collapse;font-family:arial,sans-serif;font-size:13px">Clifton Park, NY</span><div><div>Phone: (518) 881-4924 or <span style="border-collapse:collapse;color:rgb(136,136,136);font-family:arial,sans-serif;font-size:13px">(518) 371-3971 x124</span></div>
<div><font face="Verdana, Arial, Helvetica, sans-serif"><span style="line-height:31px;font-size:x-small"><br></span></font></div></div></div></div></div><br>
</div></div>