Merge lp:~afrantzis/lava-test/fix-installdeps-cmd into lp:lava-test/0.0

Proposed by Alexandros Frantzis
Status: Merged
Merged at revision: 15
Proposed branch: lp:~afrantzis/lava-test/fix-installdeps-cmd
Merge into: lp:lava-test/0.0
Diff against target: 12 lines (+1/-1)
1 file modified
abrek/testdef.py (+1/-1)
To merge this branch: bzr merge lp:~afrantzis/lava-test/fix-installdeps-cmd
Reviewer Review Type Date Requested Status
Paul Larson Pending
Review via email: mp+29850@code.launchpad.net

Description of the change

Fix creation of command string for dependency installation.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'abrek/testdef.py'
2--- abrek/testdef.py 2010-07-09 21:47:20 +0000
3+++ abrek/testdef.py 2010-07-14 09:54:43 +0000
4@@ -116,7 +116,7 @@
5 def _installdeps(self):
6 if not self.deps:
7 return 0
8- cmd = "sudo apt-get install %s", " ".join(self.deps)
9+ cmd = "sudo apt-get install %s" % " ".join(self.deps)
10 rc, output = getstatusoutput(cmd)
11 if rc:
12 raise RuntimeError("Dependency installation failed")

Subscribers

People subscribed via source and target branches