<p>I agree with both of you. It will make maintenance much easier. Maintaining one location will solve a lot of issues.</p>
<p>Andrew</p>
<p>Sent from an Android tablet.</p>
<div class="gmail_quote">On May 24, 2012 11:54 PM, "Marcus D. Hanwell" <<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I agree with Bill, and would like to pass this in for all tests. It<br>
also isn't clear to me why we don't just use a larger amount of this<br>
logic in the wrapped C++ module, that way we would only need to<br>
maintain one location where the pipeline logic is setup, along with<br>
the logic for what to do if the file is not present, where to write<br>
baselines etc.<br>
<br>
Marcus<br>
<br>
On Thu, May 24, 2012 at 9:18 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> According to Wrapping.Python/vtk/test/Testing.py<br>
> the path to the testing directory is determined by:<br>
><br>
>    -T /path/to/valid/temporary_dir/<br>
>    --temp-dir /path/to/valid/temporary_dir/<br>
><br>
>          This is a path to the directory where the image differences<br>
>          are written.  If this option is not set via the command line<br>
>          the environment variable VTK_TEMP_DIR is used.  If the<br>
>          environment variable is not set the value defaults to<br>
>          '../../../Testing/Temporary'.<br>
><br>
> For VTK6, the default should be<br>
> ../../../../Testing/Temporary<br>
><br>
> I just pushed a patch to gerrit to resolve the problem.<br>
><br>
> However, in the future I think we should explicitly specify -T in the<br>
> add_test commands for python tests. We do that for tcl and cxx tests.<br>
> It's a bit too tricky to rely on defaults in my opinion.<br>
><br>
> Bill<br>
><br>
><br>
> On Thu, May 24, 2012 at 3:47 AM, Andrew Maclean<br>
> <<a href="mailto:andrew.amaclean@gmail.com">andrew.amaclean@gmail.com</a>> wrote:<br>
>> Hi Marcus,<br>
>> I have been monitoring TestParametricFunctions and from the dashboard it<br>
>> seems that it fails because it is trying to write the test files<br>
>> to /Common/Testing/Temporary/ which does not exist. On my linux machine, if<br>
>> I create this directory then the test runs. I can confirm this on on my<br>
>> machine running Ubuntu 64-bit linux.<br>
>> I am seeing the same thing on londinium.kitware for both clang and gcc.<br>
>> It is passing on the Macs andhythloth.kitware.<br>
>> The test also runs on my windows 64-bit machine and passed.<br>
>><br>
>> I am pretty ignorant here as to whether this directory is created on the fly<br>
>> and fails because Testing does not exist in Common or whether it is<br>
>> something else. It is odd that both Mac and my windows machine run the test<br>
>> Ok.<br>
>><br>
>><br>
>> Andrew<br>
>><br>
>><br>
>><br>
>> --<br>
>> ___________________________________________<br>
>> Andrew J. P. Maclean<br>
>><br>
>> ___________________________________________<br>
><br>
><br>
><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
</blockquote></div>