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
=== modified file 'lava_test/core/installers.py'
--- lava_test/core/installers.py 2011-09-12 09:19:10 +0000
+++ lava_test/core/installers.py 2012-02-27 03:40:23 +0000
@@ -62,7 +62,7 @@
62 if self.deps:62 if self.deps:
63 if observer: observer.about_to_install_packages(self.deps)63 if observer: observer.about_to_install_packages(self.deps)
64 # XXX: Possible point of target-specific package installation64 # XXX: Possible point of target-specific package installation
65 cmd = "sudo apt-get install -y " + " ".join(self.deps)65 cmd = "sudo apt-get install -y --force-yes " + " ".join(self.deps)
66 self._run_shell_cmd(cmd, observer)66 self._run_shell_cmd(cmd, observer)
67 if observer: observer.did_install_packages(self.deps)67 if observer: observer.did_install_packages(self.deps)
6868

Subscribers

People subscribed via source and target branches