<div dir="ltr"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif">Hi,</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">
Attached is an updated version of the P4 patch.</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">
Perforce always returns absolute file paths with the depot name (//depot/path/to/file.txt) and that was breaking the test infrastructure, which expects relative path names.</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">Previously I was removing the depot name (//depot/) to make the test happy but I realized it was a bad idea now that we are looking at changelists from CDash using P4Web, as the depot needs to be added manually (which will break multi-depot repositories).</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">The new version does not strip the depot name from the files, which are passed untouched to Update.xml (that should have been the right behavior all along). CTestUpdateCommon removes a given prefix (if defined in the REPOSITORY_FILE_PREFIX variable) after reading files from Update.xml to satisfy the checks.</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 class="gmail_default" style="font-family:'trebuchet ms',sans-serif">
<br></div></div>