Merge lp://staging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages into lp://staging/ubuntu/oneiric/upstart
Proposed by
Clint Byrum
Status: | Merged |
---|---|
Merged at revision: | 1326 |
Proposed branch: | lp://staging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages |
Merge into: | lp://staging/ubuntu/oneiric/upstart |
Diff against target: |
52 lines (+36/-1) 2 files modified
debian/changelog (+8/-0) debian/conf/failsafe.conf (+28/-1) |
To merge this branch: | bzr merge lp://staging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ubuntu branches | Pending | ||
Review via email: mp+75278@code.staging.launchpad.net |
Description of the change
This attempts to make it as clear as possible to users what is blocking the system boot when they have interfaces defined in /etc/network/
To post a comment you must log in.
+ $PLYMOUTH pause-progress || :
Why does the progress need to be paused? This probably makes no difference with the standard Ubuntu plymouth theme, since we simply cycle the dots on the screen, but for themes that do care about progress bars I'm not sure pausing is the right thing. Have you tried it without this change?
+ $PLYMOUTH message --text="Waiting for network configuration..." || :
+ sleep 40
+ $PLYMOUTH message --text="Waiting up to 60 more seconds for network configuration..." || :
+ sleep 59
What's the value in having two different messages here instead of just one? Is this to keep the user from thinking the machine has hung? (If so, see above about not pausing progress...?)
+ continue_boot () {
+ $PLYMOUTH unpause-progress || :
+ }
Does this work in practice, or does upstart kill the job with too much force?
Also, I think this leaves the last message on the screen - do we want to clear it instead?
+ sleep 1
Not sure what this buys us - is this to try to ensure there's time to see the message before plymouth exits?