No legitimate reason that I can think of...<br><br>Feel free to add it in ExpandDataFileName as long as all the tests still pass. (Can't see why they wouldn't...) Just keep an eye on the dashboards after your commits. Actually, why not just make vtkTestUtilities call vtkTesting::GetDataRoot and eliminate the code duplication...?<br>
<br><br>Thanks,<br>David Cole<br><br><br><div class="gmail_quote">2008/10/30 Shepherd, Jason F <span dir="ltr"><<a href="mailto:jfsheph@sandia.gov">jfsheph@sandia.gov</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">







<div>
<font size="2" face="Arial, sans-serif">
<div>Hey guys,</div>
<div> </div>
<div>I've noticed that there are (seemingly) two ways of getting to the VTKData directory from a test file:</div>
<div> </div>
<div>vtkTestUtilities::ExpandDataFileName(…)</div>
<div>and</div>
<div>vtkTesting::GetDataRoot()</div>
<div> </div>
<div>The first one looks for a command line argument, then for a VTK_DATA_ROOT environment variable, and failing those uses "../../../../VTKData".</div>
<div>The second option looks for a command line argument, then for a VTK_DATA_ROOT environment variable, then a CMAKE variable (VTK_DATA_ROOT, again), and failing those applies "../../../../VTKData".</div>
<div> </div>
<div>Is there a reason for not checking for the CMAKE variable in ExpandDataFileName() ?</div>
<div> </div>
<div>Jason Shepherd</div>
<div> </div>
<div> </div>
</font>
</div>

<br>_______________________________________________<br>
vtk-developers mailing list<br>
<a href="mailto:vtk-developers@vtk.org">vtk-developers@vtk.org</a><br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br></blockquote></div><br>