units/console-getty.service: comment reason for ConditionPathExists
authorAlan Jenkins <alan.christopher.jenkins@gmail.com>
Wed, 9 Aug 2017 13:43:41 +0000 (14:43 +0100)
committerAlan Jenkins <alan.christopher.jenkins@gmail.com>
Wed, 9 Aug 2017 17:51:46 +0000 (18:51 +0100)
Currently we have 4 getty services.  1 has a BindsTo dependency on a
device unit.  3 have ConditionPathExists, but the reason is different in
every single one.

* Add comment to console-getty@.service (see commit 1b41981d)
* getty@.service is already commented
* container-getty.service is not strictly correct, as I realized while
  trying to compose a comment.  Reported as #6584.

units/console-getty.service.m4.in

index ecf1db1..8b6a4ab 100644 (file)
@@ -9,12 +9,14 @@
 Description=Console Getty
 Documentation=man:agetty(8) man:systemd-getty-generator(8)
 After=systemd-user-sessions.service plymouth-quit-wait.service
-ConditionPathExists=/dev/console
 m4_ifdef(`HAVE_SYSV_COMPAT',
 After=rc-local.service
 )m4_dnl
 Before=getty.target
 
+# OCI containers may be run without a console
+ConditionPathExists=/dev/console
+
 [Service]
 # The '-o' option value tells agetty to replace 'login' arguments with an
 # option to preserve environment (-p), followed by '--' for safety, and then