|Summary:||run_webkit_tests.py shouldn't have platform-specific logic|
|Product:||WebKit||Reporter:||Adam Barth <abarth>|
|Component:||New Bugs||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||cjerdonek, dpranke, eric, ojan|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
|Bug Depends on:|
Description Adam Barth 2010-04-10 08:08:45 PDT
run_webkit_tests.py shouldn't have platform-specific logic
Comment 3 Dirk Pranke 2010-04-10 11:11:25 PDT
Comment on attachment 53041 [details] Patch Looks basically good to me, although I'm really not that familiar with what the Executive package does; I'll have to look into that more now that this code is rather dependent on it.
Comment 4 Adam Barth 2010-04-10 12:43:14 PDT
Thanks. Executive.cpu_count shouldn't exist. It only exits because the multiprocessing package is in Python 2.6 and later. We shouldn't have a bunch of different pieces of code that work around the lack of the multiprocessing. In principle, we could just implement our own version of multiprocessing, like we do for os.relpath, but we decide to put it on executive so it could be non-static (which is better for testing). The main point of this patch is that it makes sense for a WebKit port to recommend a default number of DRT instances. It doesn't make a lot of sense for a WebKit port to tell you how many CPUs your machine has. Some lower-level piece of code should do that. The webkitpy.common.system package houses all the non-WebKit specific code for interacting with the operating system (that isn't supplied by the standard library), so the CPU counting code should live there. Now, what to do with the CPU count, that's a question for the layout_test package.
Comment 5 Eric Seidel (no email) 2010-04-10 14:12:17 PDT
Comment on attachment 53041 [details] Patch I wish executive had more testing. OK. We need to kill the 4-limit hack. I have never reproduced the errors Dirk saw, but I would like to... so we can fix them.
Comment 6 Adam Barth 2010-04-10 15:17:37 PDT
Comment on attachment 53041 [details] Patch Clearing flags on attachment: 53041 Committed r57422: <http://trac.webkit.org/changeset/57422>
Comment 7 Adam Barth 2010-04-10 15:17:50 PDT
All reviewed patches have been landed. Closing bug.