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.

Pedro


On Thu, Oct 24, 2013 at 2:03 PM, Pedro Navarro <pnava...@netflix.com> wrote:

> 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.
>
> About the DetectVCS refactor, I'll take a stab at it!
>
> Pedro
>
>
> On Thu, Oct 24, 2013 at 1:01 PM, Brad King <brad.k...@kitware.com> wrote:
>
>> On 10/24/2013 03:57 PM, Pedro Navarro wrote:
>> > That's right, p4 usually doesn't create a .p4 file
>>
>> I think it is fine for the test to not place .p4 files
>> since that is representative of real use cases.
>>
>> > Ideally each VCS implementation could have a way to tell
>> > CTest if a directory it's under that specific VCS control
>> > or not, that way we could execute whatever p4 commands we
>> > wanted inside cmCTestP4 and return true if the source
>> > directory is being managed by p4.
>>
>> I think that would be a great way to refactor DetectVCS.
>> It would avoid putting VCS-tool-specific knowledge (the
>> directory name) in the main class.  The existing classes
>> for other tools would just look for its control directory
>> and P4 could run the tool.
>>
>> Thanks,
>> -Brad
>>
>
>
--

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to