Merge lp:~afrantzis/ubuntu-touch-session/keep-display-on-during-boot into lp:ubuntu-touch-session

Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: 290
Merged at revision: 292
Proposed branch: lp:~afrantzis/ubuntu-touch-session/keep-display-on-during-boot
Merge into: lp:ubuntu-touch-session
Diff against target: 32 lines (+17/-0)
2 files modified
debian/ubuntu-touch-session.install (+1/-0)
keep-display-on-during-boot.conf (+16/-0)
To merge this branch: bzr merge lp:~afrantzis/ubuntu-touch-session/keep-display-on-during-boot
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Pat McGowan (community) Approve
Review via email: mp+307949@code.launchpad.net

Commit message

Keep the display on during boot (LP: #1623853)

The display-on lock is required so that the phone doesn't suspend during boot
if long-running jobs are executed before the session starts (e.g. apparmor
profile recompilation during the first boot after an upgrade).

Description of the change

Keep the display on during boot (LP: #1623853)

The display-on lock is required so that the phone doesn't suspend during boot
if long-running jobs are executed before the session starts (e.g. apparmor
profile recompilation during the first boot after an upgrade).

To post a comment you must log in.
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

I tested it and it is much better although obviously still lacks any indication things are happening. Along with a note for people to expect it to take some time on the boot screen should be ok until we can address it further.

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Looks like a terrible workaround, but better this than nothing.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/ubuntu-touch-session.install'
2--- debian/ubuntu-touch-session.install 2015-02-07 02:22:15 +0000
3+++ debian/ubuntu-touch-session.install 2016-10-07 12:25:40 +0000
4@@ -3,6 +3,7 @@
5 startup-scripts/* /usr/bin/
6 *.override /etc/init
7 factory-wipe.conf /etc/init/
8+keep-display-on-during-boot.conf /etc/init
9 systemd-overrides/* /lib/systemd/system/
10 ubuntu-touch-session.d/* /etc/ubuntu-touch-session.d/
11 upstart-session/* /usr/share/upstart/sessions/
12
13=== added file 'keep-display-on-during-boot.conf'
14--- keep-display-on-during-boot.conf 1970-01-01 00:00:00 +0000
15+++ keep-display-on-during-boot.conf 2016-10-07 12:25:40 +0000
16@@ -0,0 +1,16 @@
17+description "Keep the display on during boot"
18+author "Alexandros Frantzis <alexandros.frantzis@canonical.com>"
19+
20+# Acquire a display-on lock when repowerd has started, and release it when
21+# lightdm has started. Note that after releasing the display-on lock we get an
22+# additional 30s (at least) before the display turns off, leaving enough time
23+# for lightdm to fully start the session.
24+#
25+# The display-on lock is required so that the phone doesn't suspend during boot
26+# if long-running jobs are executed before the session starts (e.g. apparmor
27+# profile recompilation during the first boot after an upgrade, LP: #1623853).
28+
29+start on started repowerd
30+stop on started lightdm
31+
32+exec repowerd-cli display on

Subscribers

People subscribed via source and target branches