<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">CDash now supports P4Web for viewing files and diffs and the daily changes are read from a Perforce repository! I'll see what happens tonight, when the nightly builds are run and will create a thread with the patches in the CDash mailing list if succesful.</div>
<div class="gmail_default" style="font-family:trebuchet ms,sans-serif"><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Pedro</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Oct 24, 2013 at 2:03 PM, Pedro Navarro <span dir="ltr"><<a href="mailto:pnavarro@netflix.com" target="_blank">pnavarro@netflix.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Turns out that I could find a way to have P4Web do diffs based on changelists, so no changes to CTest where needed. I have P4Web now working on CDash. What's the process to submit CDash patches? I just signed up for its mailing list.</div>

<div class="gmail_default" style="font-family:trebuchet ms,sans-serif"><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">About the DetectVCS refactor, I'll take a stab at it!</div><span class="HOEnZb"><font color="#888888"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">

<br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Pedro</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 24, 2013 at 1:01 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@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>On 10/24/2013 03:57 PM, Pedro Navarro wrote:<br>
> That's right, p4 usually doesn't create a .p4 file<br>
<br>
</div>I think it is fine for the test to not place .p4 files<br>
since that is representative of real use cases.<br>
<div><br>
> Ideally each VCS implementation could have a way to tell<br>
> CTest if a directory it's under that specific VCS control<br>
> or not, that way we could execute whatever p4 commands we<br>
> wanted inside cmCTestP4 and return true if the source<br>
> directory is being managed by p4.<br>
<br>
</div>I think that would be a great way to refactor DetectVCS.<br>
It would avoid putting VCS-tool-specific knowledge (the<br>
directory name) in the main class.  The existing classes<br>
for other tools would just look for its control directory<br>
and P4 could run the tool.<br>
<br>
Thanks,<br>
-Brad<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>