diff options
| author | Filipe Brandenburger <filbranden@gmail.com> | 2020-06-17 12:17:54 -0700 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2020-06-24 16:49:26 +0200 |
| commit | 41d1f469cf10f5f3e9cb4f4853ace9b0cfe5beae (patch) | |
| tree | 5a8bbe67cfafb46b99daf93f219699f93b968a95 /src/escape | |
| parent | a1ba8c5b71164665ccb53c9cec384e5eef7d3689 (diff) | |
| download | systemd-41d1f469cf10f5f3e9cb4f4853ace9b0cfe5beae.tar.gz | |
log: introduce log_parse_environment_cli() and log_setup_cli()
Presently, CLI utilities such as systemctl will check whether they have a tty
attached or not to decide whether to parse /proc/cmdline or EFI variable
SystemdOptions looking for systemd.log_* entries.
But this check will be misleading if these tools are being launched by a
daemon, such as a monitoring daemon or automation service that runs in
background.
Make log handling of CLI tools uniform by never checking /proc/cmdline or EFI
variables to determine the logging level.
Furthermore, introduce a new log_setup_cli() shortcut to set up common options
used by most command-line utilities.
Diffstat (limited to 'src/escape')
| -rw-r--r-- | src/escape/escape.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/escape/escape.c b/src/escape/escape.c index 9066c30853..0c543a90f6 100644 --- a/src/escape/escape.c +++ b/src/escape/escape.c @@ -159,9 +159,7 @@ static int run(int argc, char *argv[]) { char **i; int r; - log_show_color(true); - log_parse_environment(); - log_open(); + log_setup_cli(); r = parse_argv(argc, argv); if (r <= 0) |