[cmake-developers] Perforce Patch for CTest

[Pedro Navarro]

Thanks a lot! Glad we could contribute.

What's the process for changes? :) Adding support for P4Web in CDash made
me realize that maybe we need to change how revisions are reported in
Update.xml (file revisions instead of changelists), I'm looking into that
right now.

Pedro


On Wed, Oct 23, 2013 at 1:49 PM, Brad King <brad.king at kitware.com> wrote:

> On 10/22/2013 06:11 PM, Pedro Navarro wrote:
> > Good point :) Here's the new patch.
>
> Great, thanks!  I've applied it:
>
>  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c63ecee3
>
> with minor tweaks:
>
> * Updated commit author date to when you sent the email
> * Wrote more extensive commit message
> * Marked P4_EXECUTABLE as an advanced cache option
> * Simplified the cygwin exclusion logic for adding the test
> * Quoted the p4d shell command args to support spaces in path
>
> I also downloaded the p4 and p4d binaries and put them on
> the hythloth.kitware dashboard machine so they should run as
> part of its Linux64-gnu test tonight.  I used them to run the
> test locally and it works for me :)
>
> Great work,
> -Brad
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20131023/3fd9270e/attachment.html>