diff options
author | Martin Ejdestig <mejdestig@luxoft.com> | 2019-07-18 10:19:08 +0200 |
---|---|---|
committer | gunnarx <gunnarx@users.noreply.github.com> | 2019-07-22 12:49:16 +0200 |
commit | aaeff31499eefdfc38b9d225050e2caefec00150 (patch) | |
tree | dd4a2e643e8155a31bd8dcfacafd0bd187ed3a6b /dbus_specifications | |
parent | 6e827a990f7e0dceeaeddb7f16c710b7cc30f2fe (diff) | |
download | persistence-common-object-aaeff31499eefdfc38b9d225050e2caefec00150.tar.gz |
Revert "dbus_config: Remove dbus configuration file"
This reverts commit 103be9f0bb77df0b139541ab366ff98a4fe4e408.
No D-Bus policy file is installed which causes persistence-administrator
to fail during startup since it ends up calling a persitence-client-object
function that wants to own the name org.genivi.persistence.admin. Owning
names is denied by default on the system bus.
Since it is persistence-common-object that takes the name
org.genivi.persistence.admin in src/pers_ipc_dbus.c:persIpcPASLoopThread(),
the configuration probably belongs in this repository.
It looks like persistence-client-library has never installed
org.genivi.persistence.admin.conf itself in its build system. It was done
for a while in meta-ivi which must have been the source of the conflict
mentioned in the reverted commit. Installed in
GENIVI/meta-ivi@7318681d851e3e3ee22a but later removed in
GENIVI/meta-ivi@9d5a45e473fbb3a29f5e with the comment "it should be done
by PCO".
The reverted commit was sent in pull request #3 with no reasoning as to why
it should be added by persistent-client-library instead of
persistence-common-object. Later pull request #5 first adds it but then
removes it again in a later commit. Unsure why. Reasoning in comment about
the missing D-Bus config file seemed correct.
Signed-off-by: Martin Ejdestig <mejdestig@luxoft.com>
Diffstat (limited to 'dbus_specifications')
0 files changed, 0 insertions, 0 deletions