Bug 12398 - patch run-webkit-tests to put Qt results in their own directory
Summary: patch run-webkit-tests to put Qt results in their own directory
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 420+
Hardware: Other Linux
: P2 Normal
Assignee: Lars Knoll
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-24 13:40 PST by Lars Knoll
Modified: 2007-01-26 09:41 PST (History)
0 users

See Also:


Attachments
diff for run-webkit-tests (9.09 KB, patch)
2007-01-24 13:40 PST, Lars Knoll
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Knoll 2007-01-24 13:40:03 PST
The patch below puts the test results for the Qt build into a directory of it's own. They will now all go into WebKit/LayoutTestResults/qt.

In addition, the patch unifies the results for text only tests (so that no additional Qt specific result file is generated for these).

I'd like to check in the test results for Qt rather soon, so I would also need a signoff to submit them :)
Comment 1 Lars Knoll 2007-01-24 13:40:52 PST
Created attachment 12647 [details]
diff for run-webkit-tests
Comment 2 Darin Adler 2007-01-24 14:04:21 PST
Cool. Ideally we'd set things up so that we could have results that are platform-independent, and an overlay of platform-dependent results for each. So we'd always search first in the platform-dependent directory and then fall back to the platform-independent if there was nothing there. Then we could make a mac platform-dependent directory and move test results into there as appropriate.

Similarly, I also think we should have platform-dependent lists of tests to skip.
Comment 3 Darin Adler 2007-01-24 14:06:00 PST
Comment on attachment 12647 [details]
diff for run-webkit-tests

In the long run, I think that some text-only tests will be platform-specific and some layout tests will be platform-independent. So while this is a good first cut I think we will need to refine this still more.

r=me
Comment 4 Darin Adler 2007-01-26 09:41:51 PST
Looks like Lars committed this in r19111.