diff options
author | Patrick Williams <patrick@stwcx.xyz> | 2017-04-26 11:14:47 -0500 |
---|---|---|
committer | Patrick Williams <patrick@stwcx.xyz> | 2017-04-26 11:14:49 -0500 |
commit | 0eb60c7b253c7d0a8b099a8787ca55fc567a7463 (patch) | |
tree | 15a7cc1cb91413dd9b8980df614e51e2a58cf554 /meta-phosphor/common/recipes-phosphor/console | |
parent | 3a0ee496742d9d11fe3ecbac507f1c59ee81e1ea (diff) | |
download | talos-openbmc-0eb60c7b253c7d0a8b099a8787ca55fc567a7463.tar.gz talos-openbmc-0eb60c7b253c7d0a8b099a8787ca55fc567a7463.zip |
obmc-console: clean up service file
An old TODO was left in and never resolved. The
ExecStart directive did some bash invocations
because older versions of the server did not support
a parameter which was a /dev alias, like /dev/ttyVUART0,
and instead always required real devices like 'ttyS2'.
Latest versions of obmc-console support ttyVUART0 as a
parameter, so use it directly.
Change-Id: I8539d7e5e65888f129486d7a79bf032f343fd4b7
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Diffstat (limited to 'meta-phosphor/common/recipes-phosphor/console')
-rw-r--r-- | meta-phosphor/common/recipes-phosphor/console/obmc-console/obmc-console@.service | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/meta-phosphor/common/recipes-phosphor/console/obmc-console/obmc-console@.service b/meta-phosphor/common/recipes-phosphor/console/obmc-console/obmc-console@.service index c50b3b1b4..e28b06933 100644 --- a/meta-phosphor/common/recipes-phosphor/console/obmc-console/obmc-console@.service +++ b/meta-phosphor/common/recipes-phosphor/console/obmc-console/obmc-console@.service @@ -4,8 +4,7 @@ BindsTo=dev-%i.device After=dev-%i.device [Service] -# TODO: openbmc/obmc-console#5 - Restore this to 'obmc-console-server ttyVUART0' -ExecStart=/bin/sh -c 'realpath /dev/%i | xargs basename | xargs {sbindir}/obmc-console-server --config {sysconfdir}/obmc-console.conf' +ExecStart={sbindir}/obmc-console-server --config {sysconfdir}/obmc-console.conf %i Restart=always [Install] |