Merge lp:~qzhang/lava-test/fix-941737 into lp:lava-test/0.0

Proposed by Spring Zhang
Status: Merged
Merged at revision: 120
Proposed branch: lp:~qzhang/lava-test/fix-941737
Merge into: lp:lava-test/0.0
Diff against target: 11 lines (+1/-1)
1 file modified
lava_test/core/installers.py (+1/-1)
To merge this branch: bzr merge lp:~qzhang/lava-test/fix-941737
Reviewer Review Type Date Requested Status
Paul Larson (community) Approve
Review via email: mp+94711@code.launchpad.net

Description of the change

add option --force-yes to avoid error sometimes

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Yes, let's merge this. Please see comments in the bug.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lava_test/core/installers.py'
2--- lava_test/core/installers.py 2011-09-12 09:19:10 +0000
3+++ lava_test/core/installers.py 2012-02-27 03:40:23 +0000
4@@ -62,7 +62,7 @@
5 if self.deps:
6 if observer: observer.about_to_install_packages(self.deps)
7 # XXX: Possible point of target-specific package installation
8- cmd = "sudo apt-get install -y " + " ".join(self.deps)
9+ cmd = "sudo apt-get install -y --force-yes " + " ".join(self.deps)
10 self._run_shell_cmd(cmd, observer)
11 if observer: observer.did_install_packages(self.deps)
12

Subscribers

People subscribed via source and target branches