Merge ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:uaudit-fix-install-options into ubuntu-security-tools:master

Proposed by Spyros Seimenis
Status: Merged
Merged at revision: 4ccf49fac01c52d2b3df1cf13703dfabea7317c7
Proposed branch: ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:uaudit-fix-install-options
Merge into: ubuntu-security-tools:master
Diff against target: 18 lines (+5/-2)
1 file modified
audits/uaudit (+5/-2)
Reviewer Review Type Date Requested Status
Alex Murray Approve
Review via email: mp+442823@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Murray (alexmurray) wrote :

LGTM - thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/audits/uaudit b/audits/uaudit
index 09cf3df..9fa9db7 100755
--- a/audits/uaudit
+++ b/audits/uaudit
@@ -72,8 +72,11 @@ class StaticAnalysisTool(object):
72 self._install_options = install_options72 self._install_options = install_options
7373
74 def install_cmd(self) -> list:74 def install_cmd(self) -> list:
75 return ["sudo", "apt" if self._source == StaticAnalysisToolSource.DEB else "snap",75 cmd = ["sudo", "apt" if self._source == StaticAnalysisToolSource.DEB else "snap",
76 "install", self._install_options, self.name]76 "install", self.name]
77 if self._install_options != "":
78 cmd.append(self._install_options)
79 return cmd
7780
78 def exec_cmd(self, format) -> list:81 def exec_cmd(self, format) -> list:
79 if format == "txt":82 if format == "txt":

Subscribers

People subscribed via source and target branches