<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><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 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 class="im">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 class="im"><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>