powerd

Merge lp:~sforshee/powerd/fix-power-button into lp:powerd

  1. fix-power-button
  2. Merge into trunk
Proposed by Seth Forshee
Status: Merged
Approved by: Ricardo Salveti
Approved revision: 69
Merged at revision: 67
Proposed branch: lp:~sforshee/powerd/fix-power-button
Merge into: lp:powerd
Diff against target: 10 lines (+1/-0)
1 file modified
debian/upstart (+1/-0)
To merge this branch: bzr merge lp:~sforshee/powerd/fix-power-button
Reviewer Review Type Date Requested Status
Ricardo Salveti (community) Approve
PS Jenkins bot continuous-integration Approve
Oliver Grawert Approve
Review via email: mp+174203@code.launchpad.net

Commit message

Add ANDROID_ROOT=/system to upstart job to get power button working again (LP: 1200156)

Description of the change

Add ANDROID_ROOT=/system to upstart job to get power button working again (LP: 1200156)

To post a comment you must log in.
Revision history for this message
Matt Fischer (mfisch) wrote :

+1

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:67
http://jenkins.qa.ubuntu.com/job/powerd-ci/88/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/42
        deb: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/42/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/powerd-ci/88/rebuild

review: Approve (continuous-integration)
lp:~sforshee/powerd/fix-power-button updated
69. By Seth Forshee

Remove QT_QPA_PLATFORM from upstart job again

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://jenkins.qa.ubuntu.com/job/powerd-autolanding/53/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-autolanding/29
        deb: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-autolanding/29/artifact/work/output/*zip*/output.zip

review: Needs Fixing (continuous-integration)
Revision history for this message
Oliver Grawert (ogra) wrote :

looks good now, verified to work on maguro

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:69
http://jenkins.qa.ubuntu.com/job/powerd-ci/89/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/43
        deb: http://jenkins.qa.ubuntu.com/job/powerd-saucy-armhf-ci/43/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/powerd-ci/89/rebuild

review: Approve (continuous-integration)
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/upstart'
2--- debian/upstart 2013-07-10 17:56:54 +0000
3+++ debian/upstart 2013-07-11 14:42:25 +0000
4@@ -4,6 +4,7 @@
5
6 respawn
7
8+env ANDROID_ROOT=/system
9 #uncomment the line below to enable debugging
10 #env POWERD_DEBUG=1
11

Subscribers

People subscribed via source and target branches