QtDeclarative bindings for Powerd

Merge lp:~ricmm/qtpowerd/use-pid-for-name into lp:qtpowerd

  1. use-pid-for-name
  2. Merge into trunk
Proposed by Ricardo Mendoza
Status: Merged
Approved by: Loïc Minier
Approved revision: 8
Merged at revision: 8
Proposed branch: lp:~ricmm/qtpowerd/use-pid-for-name
Merge into: lp:qtpowerd
Diff against target: 11 lines (+1/-1)
1 file modified
QtPowerd.cpp (+1/-1)
To merge this branch: bzr merge lp:~ricmm/qtpowerd/use-pid-for-name
Reviewer Review Type Date Requested Status
Loïc Minier Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+191052@code.launchpad.net

Commit message

Use pid to construct lock name

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:8
http://jenkins.qa.ubuntu.com/job/qtpowerd-ci/3/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtpowerd-saucy-amd64-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtpowerd-saucy-armhf-ci/3

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/qtpowerd-ci/3/rebuild

review: Approve (continuous-integration)
Revision history for this message
Loïc Minier (lool) wrote :

Binaries provided by Ricardo Mendoza fix mediaplayer-app for me with this branch.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'QtPowerd.cpp'
2--- QtPowerd.cpp 2013-10-14 18:53:23 +0000
3+++ QtPowerd.cpp 2013-10-14 21:02:53 +0000
4@@ -46,7 +46,7 @@
5 "com.canonical.powerd",
6 QDBusConnection::SM_BUSNAME(), this);
7
8- m_lockName = QGuiApplication::instance()->applicationName();
9+ m_lockName = QString::number(QGuiApplication::instance()->applicationPid());
10 m_lockName.append("-background");
11 }
12

Subscribers

People subscribed via source and target branches