Merge lp://staging/~phablet-team/history-service/single_instance_using_lock_file into lp://staging/history-service/staging
Proposed by
Gustavo Pichorim Boiko
Status: | Merged |
---|---|
Approved by: | Tiago Salem Herrmann |
Approved revision: | 277 |
Merged at revision: | 262 |
Proposed branch: | lp://staging/~phablet-team/history-service/single_instance_using_lock_file |
Merge into: | lp://staging/history-service/staging |
Prerequisite: | lp://staging/~phablet-team/history-service/optimize_dbus_traffic |
Diff against target: |
47 lines (+16/-5) 2 files modified
cmake/modules/GenerateTest.cmake (+1/-0) daemon/main.cpp (+15/-5) |
To merge this branch: | bzr merge lp://staging/~phablet-team/history-service/single_instance_using_lock_file |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tiago Salem Herrmann (community) | Approve | ||
Review via email: mp+318949@code.staging.launchpad.net |
Commit message
Use a QLockFile to ensure there will be only one instance of the daemon per user.
As we now delay the registration on dbus, sometimes we ended up having two instances of the daeon running (because of dbus activation). This change makes sure that won't happen.
Description of the change
Use a QLockFile to ensure there will be only one instance of the daemon per user.
As we now delay the registration on dbus, sometimes we ended up having two instances of the daeon running (because of dbus activation). This change makes sure that won't happen.
To post a comment you must log in.
Looks good to me.
good to have this finally resolved.