Title: [164727] trunk/Tools
Revision
164727
Author
[email protected]
Date
2014-02-26 11:16:21 -0800 (Wed, 26 Feb 2014)

Log Message

runscript: line 42: /bin/ls: Argument list too long
https://bugs.webkit.org/show_bug.cgi?id=129376

Reviewed by Michael Saboff.

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.

* Scripts/jsc-stress-test-helpers/shell-runner.sh:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (164726 => 164727)


--- trunk/Tools/ChangeLog	2014-02-26 19:05:16 UTC (rev 164726)
+++ trunk/Tools/ChangeLog	2014-02-26 19:16:21 UTC (rev 164727)
@@ -1,3 +1,16 @@
+2014-02-26  Mark Hahnenberg  <[email protected]>
+
+        runscript: line 42: /bin/ls: Argument list too long
+        https://bugs.webkit.org/show_bug.cgi?id=129376
+
+        Reviewed by Michael Saboff.
+
+        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.
+
+        * Scripts/jsc-stress-test-helpers/shell-runner.sh: 
+
 2014-02-26  Martin Robinson  <[email protected]>
 
         ENABLE_PROMISES is improperly specified as ENABLE_PROMISE in Features.pm

Modified: trunk/Tools/Scripts/jsc-stress-test-helpers/shell-runner.sh (164726 => 164727)


--- trunk/Tools/Scripts/jsc-stress-test-helpers/shell-runner.sh	2014-02-26 19:05:16 UTC (rev 164726)
+++ trunk/Tools/Scripts/jsc-stress-test-helpers/shell-runner.sh	2014-02-26 19:16:21 UTC (rev 164727)
@@ -39,7 +39,7 @@
 trap "kill -9 0" SIGINT SIGHUP SIGTERM
 
 echo 0 > ${indexFile}
-ls test_script_* > ${testList}
+find . -name 'test_script_*' -depth 1 > ${testList}
 
 function lock_test_list() {
     until mkdir ${lockDir} 2> /dev/null; do sleep 0; done
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to