Depends on dbus service rather than socket #41
Conversation
With newer dbus depending on dbus.socket doesn't work and block the boot.
|
tested on jolla phone internal release candidate, no regressions |
|
What do you mean by new dbus? |
|
The issue was reproduced with Maui that has dbus 1.8.6 (and in general newer packages of a lot of stuff including systemd), but sledges now tested this patch on Jolla and it doesn't appear to break anything. |
|
Sensorfw is mer package and is designed to work with dbus and systemd from mer. There is a special dbus.socket to allow sensord start earlier than dbus.service. This will affect on boot order if we change this from .socket to .service |
| @@ -1,7 +1,7 @@ | |||
| [Unit] | |||
| Description=Sensor daemon for sensor framework | |||
| After=boardname.service | |||
| Requires=dbus.socket | |||
| Requires=dbus.service | |||
There was a problem hiding this comment.
Instead of this we should do probably Wants=dbus.service, then sensorfw could start before dbus.service is started and start initializing things. Yes, it blocks until dbus.service is up but that is the purpose as well, to allow different services to start side by side before others are ready to allow faster bootup.
|
Also we should add /usr/share/dbus-1/system-services/com.nokia.SensorService.service file that defines SystemdService=sensord.service among other things and in the systemd service file should define: BusName=com.nokia.SensorService |
|
I created new pull req and it should fix this problem also: |
|
Are both of these needed? |
|
No, this is not needed, only pull request #42 |
|
I'll close this, because pull req #42 is in |
With newer dbus depending on dbus.socket doesn't work and block the boot.