systemctl: do not suggest passing --all if the user passed --state=
authorAlan Jenkins <alan.christopher.jenkins@gmail.com>
Tue, 4 Dec 2018 17:31:51 +0000 (17:31 +0000)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 15 Jul 2019 11:37:29 +0000 (13:37 +0200)
commit8eb0cafeb0aa6687e02f45e92e60fdbad7c485c0
tree4fcb96c8f28095703c017ef0804256443b890b33
parent0f44806d8e1e0a8348d6c28951b0ed0b4cae963a
systemctl: do not suggest passing --all if the user passed --state=

"systemctl --failed" suggested I pass "--all" to see units in the inactive
state as well.  I thought this was not very useful.  If you explicitly
asked for units in a specific state, then you already know you have
narrowed it down.  And if you ran "systemctl --state=inactive", it is even
more strange to see this message.

@keszybz suggests we probably don't want to suggest "list-unit-files"
either :-).  Let's only suggest that if the user passed "--state=inactive".

Finally, this means the output for "systemctl --failed" could be just
"0 loaded units listed".  In this case, we don't need any highlight on that
text, to distinguish it from the hint.  This matches "list-unit-files".
This also means we happen to avoid using red highlight, when there are zero
failed units, as if that itself was a failure.  @kesbyz pointed out that
old behaviour was a bit weird.
src/systemctl/systemctl.c