Code review comment for lp:~mterry/snappy/systemd-restart

Revision history for this message
Martin Pitt (pitti) wrote :

LGTM.

For D-Bus services, these indeed don't have an [Install], but Type=dbus and BusName=org.bla.foo instead. For those we indeed shouldn't have a Restart=, but having it is not the end of the world (mostly just an optimization problem).

review: Approve

« Back to merge proposal