Skip to content

Commit

Permalink
Add missing comma in earlyoom argument list
Browse files Browse the repository at this point in the history
Without the comma, the string literals are concatenated.

Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python`
with a missing space in Taskcluster logs.

This was introduced in #15724.
  • Loading branch information
foolip committed Aug 23, 2019
1 parent d8cebfc commit 9058fe1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/ci/run_tc.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ def get_parser():
def start_userspace_oom_killer():
# Start userspace OOM killer: https://github.com/rfjakob/earlyoom
# It will report memory usage every minute and prefer to kill browsers.
start(["sudo", "earlyoom", "-p", "-r", "60" "--prefer=(chrome|firefox)", "--avoid=python"])
start(["sudo", "earlyoom", "-p", "-r", "60", "--prefer=(chrome|firefox)", "--avoid=python"])


def make_hosts_file():
Expand Down

0 comments on commit 9058fe1

Please sign in to comment.