The output is like this: WebKitTools/Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatch.pl ............ ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/generatePatchCommand.pl ......... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/mergeChangeLogs.pl .............. Dubious, test returned 25 (wstat 6400, 0x1900) Failed 8/16 subtests List form of pipe open not implemented at D:/google-windows-2/chromium-win-release-tests/build/WebKitTools/Scripts/VCSUtils.pm line 1579. # Looks like you planned 16 tests but ran 8. # Looks like your test exited with 25 just after 8. WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl .................... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl .............. ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl ........... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parsePatch.pl ................... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffFooter.pl ........... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl ........... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseSvnProperty.pl ............. ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl ........ ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/prepareParsedPatch.pl ........... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/removeEOL.pl .................... ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/runPatchCommand.pl .............. ok WebKitTools/Scripts/webkitperl/VCSUtils_unittest/setChangeLogDateAndReviewer.pl .. ok Test Summary Report ------------------- WebKitTools/Scripts/webkitperl/VCSUtils_unittest/mergeChangeLogs.pl (Wstat: 6400 Tests: 8 Failed: 0) Non-zero exit status: 25 Parse errors: Bad plan. You planned 16 tests but ran 8. Files=15, Tests=301, 2 wallclock secs ( 0.09 usr + 0.02 sys = 0.11 CPU) Result: FAIL Failed 1/15 test programs. 0/301 subtests failed.
Created attachment 104868 [details] Patch
Comment on attachment 104868 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104868&action=review > Tools/Scripts/VCSUtils.pm:1699 > - open(DIFF, "-|", qw(diff -u -a --binary), $fileOlder, $fileMine) or die $!; > + open(DIFF, "diff -u -a --binary $fileOlder $fileMine |") or die $!; You now need to quote $fileOlder and $fileMine in case they contain spaces.
Comment on attachment 104868 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104868&action=review >> Tools/Scripts/VCSUtils.pm:1699 >> + open(DIFF, "diff -u -a --binary $fileOlder $fileMine |") or die $!; > > You now need to quote $fileOlder and $fileMine in case they contain spaces. We don't do correct escaping on different places. e.g. only 8 lines down: http://trac.webkit.org/browser/trunk/Tools/Scripts/VCSUtils.pm#L1707
Comment on attachment 104868 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104868&action=review >>> Tools/Scripts/VCSUtils.pm:1699 >>> + open(DIFF, "diff -u -a --binary $fileOlder $fileMine |") or die $!; >> >> You now need to quote $fileOlder and $fileMine in case they contain spaces. > > We don't do correct escaping on different places. e.g. only 8 lines down: http://trac.webkit.org/browser/trunk/Tools/Scripts/VCSUtils.pm#L1707 We should fix those places (separately). We want WebKit scripts to work regardless of the path to your checkout.
Created attachment 104872 [details] Patch
Comment on attachment 104872 [details] Patch Clearing flags on attachment: 104872 Committed r93623: <http://trac.webkit.org/changeset/93623>
All reviewed patches have been landed. Closing bug.