diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-03-03 13:35:40 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-04-01 12:46:24 +0200 |
commit | 9e53c10a0f64bc1ab70877c245d5a8e08c509575 (patch) | |
tree | ea9f943a70e8b36df51d5d35e1ad79f549a067ef /src/core/job.c | |
parent | 679b0b0a21cbc14cc226c83ebe94ab2844340bc3 (diff) | |
download | systemd-9e53c10a0f64bc1ab70877c245d5a8e08c509575.tar.gz |
Flagsify EscapeStyle and make ESCAPE_BACKSLASH_ONELINE implicit
I want to tweak behaviour further, and that'll be easier when "style"
is converted to a bitfield.
Some callers used ESCAPE_BACKSLASH_ONELINE, and others not. But the
ones that didn't, simply didn't care, because the argument was assumed to
be one-line anyway (e.g. a service name). In environment-generator, this
could make a difference. But I think it's better to escape the newlines
there too. So newlines are now always escaped, to simplify the code and
the test matrix.
Diffstat (limited to 'src/core/job.c')
-rw-r--r-- | src/core/job.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/job.c b/src/core/job.c index 56c99f93eb..639dc352a6 100644 --- a/src/core/job.c +++ b/src/core/job.c @@ -846,7 +846,7 @@ static void job_print_done_status_message(Unit *u, JobType t, JobResult result) if (t == JOB_START && result == JOB_FAILED) { _cleanup_free_ char *quoted; - quoted = shell_maybe_quote(u->id, ESCAPE_BACKSLASH); + quoted = shell_maybe_quote(u->id, 0); manager_status_printf(u->manager, STATUS_TYPE_NORMAL, NULL, "See 'systemctl status %s' for details.", strna(quoted)); } } |