Bug 124846 - Disable JSC stress tests on all but Apple-Mac platforms
Summary: Disable JSC stress tests on all but Apple-Mac platforms
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Hahnenberg
URL:
Keywords:
Depends on:
Blocks: 120809
  Show dependency treegraph
 
Reported: 2013-11-25 10:19 PST by Mark Hahnenberg
Modified: 2013-12-02 13:43 PST (History)
1 user (show)

See Also:


Attachments
Patch (1.42 KB, patch)
2013-11-25 10:22 PST, Mark Hahnenberg
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Hahnenberg 2013-11-25 10:19:31 PST
Recent changes to how run-jsc-stress-tests works have made it so other platforms can't run any JSC tests. We should disable run-jsc-stress-tests on those platforms until they have ported the changes to their platforms. For now they'll fall back on the old run-javascriptcore-tests harness.
Comment 1 Mark Hahnenberg 2013-11-25 10:22:32 PST
Created attachment 217815 [details]
Patch
Comment 2 Zan Dobersek 2013-12-02 13:42:47 PST
The GTK port is currently running the stress tests without problems.
http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/42895/steps/jscore-test/logs/stdio

Is this patch still valid? Does it depend on any future patch?
If so, can you please provide some pointers, especially on what's required to re-enable the stress tests in the future if they get disabled when this lands?
Comment 3 Mark Hahnenberg 2013-12-02 13:43:41 PST
(In reply to comment #2)
> The GTK port is currently running the stress tests without problems.
> http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/42895/steps/jscore-test/logs/stdio
> 
> Is this patch still valid? Does it depend on any future patch?
> If so, can you please provide some pointers, especially on what's required to re-enable the stress tests in the future if they get disabled when this lands?

Sorry, no longer needed. I retrofitted the other patch so that it works on other platforms.