diff options
author | Lennart Poettering <lennart@poettering.net> | 2021-09-29 10:30:17 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-10-12 17:11:30 +0200 |
commit | 3636724012003a2f9b45eb9f8991ae93661de462 (patch) | |
tree | ad98b698ee9add4b0c52d441b3b8c28a95e4d96f | |
parent | 4fe53426991e829c0add9378d91f3677a23076fd (diff) | |
download | systemd-3636724012003a2f9b45eb9f8991ae93661de462.tar.gz |
core: drop "const" from NeedsDaemonReload unit dbus property
It's not "const", it can change any time if people change the fs, and we
don#t send out notifications for it. Hence don't claim it was const.
(Otherwise clients might cache it, but they should not)
Prompted-by: #20792
(cherry picked from commit e30a3ba16a5da4b46cd6c8bc363dbccb791fc45e)
-rw-r--r-- | man/org.freedesktop.systemd1.xml | 2 | ||||
-rw-r--r-- | src/core/dbus-unit.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/man/org.freedesktop.systemd1.xml b/man/org.freedesktop.systemd1.xml index c14c5b6601..eb5245b62d 100644 --- a/man/org.freedesktop.systemd1.xml +++ b/man/org.freedesktop.systemd1.xml @@ -1711,7 +1711,7 @@ node /org/freedesktop/systemd1/unit/avahi_2ddaemon_2eservice { readonly s OnFailureJobMode = '...'; @org.freedesktop.DBus.Property.EmitsChangedSignal("const") readonly b IgnoreOnIsolate = ...; - @org.freedesktop.DBus.Property.EmitsChangedSignal("const") + @org.freedesktop.DBus.Property.EmitsChangedSignal("false") readonly b NeedDaemonReload = ...; @org.freedesktop.DBus.Property.EmitsChangedSignal("false") readonly as Markers = ['...', ...]; diff --git a/src/core/dbus-unit.c b/src/core/dbus-unit.c index 124b9e496b..fe320f1b05 100644 --- a/src/core/dbus-unit.c +++ b/src/core/dbus-unit.c @@ -909,7 +909,7 @@ const sd_bus_vtable bus_unit_vtable[] = { SD_BUS_PROPERTY("OnSuccessJobMode", "s", property_get_job_mode, offsetof(Unit, on_success_job_mode), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("OnFailureJobMode", "s", property_get_job_mode, offsetof(Unit, on_failure_job_mode), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("IgnoreOnIsolate", "b", bus_property_get_bool, offsetof(Unit, ignore_on_isolate), SD_BUS_VTABLE_PROPERTY_CONST), - SD_BUS_PROPERTY("NeedDaemonReload", "b", property_get_need_daemon_reload, 0, SD_BUS_VTABLE_PROPERTY_CONST), + SD_BUS_PROPERTY("NeedDaemonReload", "b", property_get_need_daemon_reload, 0, 0), SD_BUS_PROPERTY("Markers", "as", property_get_markers, offsetof(Unit, markers), 0), SD_BUS_PROPERTY("JobTimeoutUSec", "t", bus_property_get_usec, offsetof(Unit, job_timeout), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("JobRunningTimeoutUSec", "t", bus_property_get_usec, offsetof(Unit, job_running_timeout), SD_BUS_VTABLE_PROPERTY_CONST), |