From a29271926ad5530276cec486c93ea72ef71a652c Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 5 Sep 2012 23:51:19 -0700 Subject: [PATCH] man: document that ExecStart= doesn't understand shell command lines --- TODO | 6 ------ man/systemd.service.xml | 11 ++++++++++- 2 files changed, 10 insertions(+), 7 deletions(-) diff --git a/TODO b/TODO index 326edb0af8..4519037ae7 100644 --- a/TODO +++ b/TODO @@ -83,14 +83,10 @@ Features: * currently system services appear not to generate core dumps... -* introduce /run/kmsg in containers? - * wall messages for shutdown should move to logind * allow writing multiple conditions in unit files on one line -* journal: json output needs to be able to deal with multiple assignments of the same field - * There's something wrong with escaping unit names: http://lists.freedesktop.org/archives/systemd-devel/2012-August/006292.html * cleanup ellipsation for log output in journalctl and systemctl status: have a sane way to disable ellipsation, and disable it by default when invoked in less/more @@ -103,8 +99,6 @@ Features: * maybe make systemd-detect-virt suid? or use fscaps? -* man: document in ExecStart= explicitly that we don't take shell command lines, only executable names with arguments - * shutdown: don't read-only mount anything when running in container * nspawn: --read-only is not applied recursively to submounts diff --git a/man/systemd.service.xml b/man/systemd.service.xml index 72b67c6988..c547948c6e 100644 --- a/man/systemd.service.xml +++ b/man/systemd.service.xml @@ -356,7 +356,16 @@ argument (i.e. the program to execute) may not be a variable, and must be a literal and absolute path - name. + name. + + Note that this setting does not + directly support shell command + lines. If shell command lines are to + be used they need to be passed + explicitly to a shell implementation + of some kind. Example: + ExecStart=/bin/sh -c 'dmesg | tac' +