Merge lp://staging/~dpniel/ubiquity/fix-runner into lp://staging/ubiquity

Proposed by Dan Chapman 
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 6316
Merged at revision: 6321
Proposed branch: lp://staging/~dpniel/ubiquity/fix-runner
Merge into: lp://staging/ubiquity
Diff against target: 72 lines (+40/-14)
2 files modified
autopilot/ubiquity-autopilot-runner/custom-installation/iso-override/usr/local/bin/run-autopilot.sh (+31/-14)
autopilot/ubiquity-autopilot-runner/custom-installation/iso-override/usr/share/mate/autostart/autopilot.desktop (+9/-0)
To merge this branch: bzr merge lp://staging/~dpniel/ubiquity/fix-runner
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Max Brustkern (community) Needs Fixing
Review via email: mp+270247@code.staging.launchpad.net

Description of the change

Fixes the autopilot test runner for Ubuntu-GNOME and Ubuntu-MATE

To post a comment you must log in.
6314. By Dan Chapman 

Remove --debug flag from run-autopilot.sh

Revision history for this message
Max Brustkern (nuclearbob) wrote :

This code appears to break the runner. I get a missing ;; in the logs. I can try proposing a patch after I test the other merge proposals.

review: Needs Fixing
6315. By Dan Chapman 

Commit missing ;; in run-autopilot.sh

Revision history for this message
Dan Chapman  (dpniel) wrote :

Ooops!

This should be fixed now.

6316. By Dan Chapman 

Seems I missed this when seperating my patches for commiting.

Sorry about that

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: