RESOLVED FIXED216323
kill-old-processes: Avoid calling kill command with empty string with GNU xargs 
https://bugs.webkit.org/show_bug.cgi?id=216323
Summary kill-old-processes: Avoid calling kill command with empty string with GNU xargs
Lauro Moura
Reported 2020-09-09 13:35:07 PDT
Currently, the kill-old-processes command to kill python scripts (like run-webkit-tests) has an unchecked "xargs kill" at the end, which may output an error message if there is no such command to be killed (as it is the case most of the time). Linux' version of xargs has the "-r" option which can be used to avoid running the command.
Attachments
Patch (1.56 KB, patch)
2020-09-09 13:38 PDT, Lauro Moura 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch for landing (1.58 KB, patch)
2020-09-09 19:33 PDT, Lauro Moura 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Lauro Moura
Comment 1 2020-09-09 13:38:02 PDT
Created attachment 408362 [details] Patch
Darin Adler
Comment 2 2020-09-09 14:06:23 PDT
Comment on attachment 408362 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=408362&action=review > Tools/ChangeLog:3 > + kill-old-processes: Avoid calling kill command with empty string This is a platform-specific bug and fix. On BSD systems like macOS, xargs already doesn’t do this. But on platforms with GNU xargs this "-r" option is needed.
Lauro Moura
Comment 3 2020-09-09 19:33:53 PDT
Created attachment 408402 [details] Patch for landing
Lauro Moura
Comment 4 2020-09-09 19:35:37 PDT
Committed r266808: <https://trac.webkit.org/changeset/266808>
Radar WebKit Bug Importer
Comment 5 2020-09-09 19:36:20 PDT
<rdar://problem/68612810>
Diego Pino
Comment 6 2020-09-13 22:24:57 PDT
*** Bug 215999 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.