Summary: | run-api-tests fails on chromium-win bot | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xianzhu Wang <wangxianzhu> | ||||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, rniwa, tony, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Xianzhu Wang
2011-09-29 19:24:11 PDT
Created attachment 109242 [details]
patch
Created attachment 109244 [details]
patch (fixed a grammar error in help message)
Comment on attachment 109244 [details] patch (fixed a grammar error in help message) View in context: https://bugs.webkit.org/attachment.cgi?id=109244&action=review > Tools/Scripts/run-api-tests:68 > + --[no-]build Build (or do not build) unit tests prior to running (default: $buildDefault. No use for Chromium as buile-webkit builds everything) Not used for Chromium because build-webkit builds everything Actually, I'd skip this part of the change. > Tools/Scripts/run-api-tests:85 > -buildTestTool() if $build; > +buildTestTool() if $build && !isChromium(); I'd also skip this part. > Tools/Scripts/run-api-tests:285 > - my $buildProcess = open3($childIn, $childOut, $childErr, $pathToBuildTestTool, @args) or die "Failed to run " . $buildTestTool; > + my $buildProcess = open3($childIn, $childOut, $childErr, "perl", $pathToBuildTestTool, @args) or die "Failed to run " . $buildTestTool; This is the only part that's needed. Are you sure you don't need to put $pathToBuildTestTool into @args? I'm somewhat of a Perl novice. :) Comment on attachment 109244 [details] patch (fixed a grammar error in help message) View in context: https://bugs.webkit.org/attachment.cgi?id=109244&action=review >> Tools/Scripts/run-api-tests:68 >> + --[no-]build Build (or do not build) unit tests prior to running (default: $buildDefault. No use for Chromium as buile-webkit builds everything) > > Not used for Chromium because build-webkit builds everything > > Actually, I'd skip this part of the change. Done. >> Tools/Scripts/run-api-tests:85 >> +buildTestTool() if $build && !isChromium(); > > I'd also skip this part. Done. >> Tools/Scripts/run-api-tests:285 >> + my $buildProcess = open3($childIn, $childOut, $childErr, "perl", $pathToBuildTestTool, @args) or die "Failed to run " . $buildTestTool; > > This is the only part that's needed. > > Are you sure you don't need to put $pathToBuildTestTool into @args? I'm somewhat of a Perl novice. :) I saw the same usage in old-run-webkit-tests (line 1444), and it works at least on Linux. I'm Perl novice too :) Created attachment 109249 [details]
patch v3
Comment on attachment 109249 [details]
patch v3
Thanks!
Comment on attachment 109249 [details] patch v3 Clearing flags on attachment: 109249 Committed r96388: <http://trac.webkit.org/changeset/96388> All reviewed patches have been landed. Closing bug. |