Bug 129376

Summary: runscript: line 42: /bin/ls: Argument list too long
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Mark Hahnenberg 2014-02-26 10:19:52 PST
This occurs on line 42 in jsc-stress-test-helpers/shell-runner.sh. The "test_script_*" can expand to more things than can be passed as arguments on the command line. We should use find instead, which generates the list of files rather than expanding them all on the command line and spitting them back out.
Comment 1 Mark Hahnenberg 2014-02-26 10:20:06 PST
<rdar://problem/16169656>
Comment 2 Mark Hahnenberg 2014-02-26 10:22:39 PST
Created attachment 225264 [details]
Patch
Comment 3 Michael Saboff 2014-02-26 10:25:00 PST
Comment on attachment 225264 [details]
Patch

r=me
Comment 4 WebKit Commit Bot 2014-02-26 11:11:58 PST
Comment on attachment 225264 [details]
Patch

Clearing flags on attachment: 225264

Committed r164727: <http://trac.webkit.org/changeset/164727>
Comment 5 WebKit Commit Bot 2014-02-26 11:12:00 PST
All reviewed patches have been landed.  Closing bug.