Bug 55936

Summary: old-run-webkit-tests should ignore test files used by reftests
Product: WebKit Reporter: Hayato Ito <hayato>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: dpranke, hamaji, ojan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on: 67936    
Bug Blocks: 36065    
Attachments:
Description Flags
ignore-reftests ojan: review+

Description Hayato Ito 2011-03-08 05:03:12 PST 
Please see the master bug for the rationale:
https://bugs.webkit.org/show_bug.cgi?id=36065

We are updating new-run-webkit-tests to support reftests. A old-run-webkit-tests should ignore test files used by reftests to avoid unexpected errors.
Comment 1 Hayato Ito 2011-03-08 19:23:58 PST 
Created attachment 85126 [details]
ignore-reftests
Comment 2 Ojan Vafai 2011-03-08 23:07:42 PST 
Comment on attachment 85126 [details]
ignore-reftests

View in context: https://bugs.webkit.org/attachment.cgi?id=85126&action=review

Might want to wait until bug 55457 is r+'ed to make sure that none of the paths will change.

> Tools/Scripts/old-run-webkit-tests:2460
> +

extra line break
Comment 3 Hayato Ito 2011-03-09 03:33:35 PST 
Hi Ojan,

Thank you for the review. I'll fix the extra line break and land this after 55457 is r+'ed.

(In reply to comment #2)
> (From update of attachment 85126 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=85126&action=review
> 
> Might want to wait until bug 55457 is r+'ed to make sure that none of the paths will change.
> 
> > Tools/Scripts/old-run-webkit-tests:2460
> > +
> 
> extra line break
Comment 4 Hayato Ito 2011-03-16 00:51:38 PDT 
Committed r81225: <http://trac.webkit.org/changeset/81225>