Merge lp:~zyga/checkbox/fix-1443895 into lp:checkbox

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Maciej Kisielewski
Approved revision: 3684
Merged at revision: 3686
Proposed branch: lp:~zyga/checkbox/fix-1443895
Merge into: lp:checkbox
Diff against target: 26 lines (+7/-1)
1 file modified
checkbox-ng/checkbox_ng/tools.py (+7/-1)
To merge this branch: bzr merge lp:~zyga/checkbox/fix-1443895
Reviewer Review Type Date Requested Status
Maciej Kisielewski Approve
Review via email: mp+256155@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Maciej Kisielewski (kissiel) wrote :

LGTM, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox-ng/checkbox_ng/tools.py'
2--- checkbox-ng/checkbox_ng/tools.py 2014-12-12 12:04:55 +0000
3+++ checkbox-ng/checkbox_ng/tools.py 2015-04-14 14:20:49 +0000
4@@ -26,6 +26,7 @@
5
6 from plainbox.impl.clitools import SingleCommandToolMixIn
7 from plainbox.impl.clitools import ToolBase
8+from plainbox.impl.providers import special
9 from plainbox.impl.providers.v1 import all_providers
10 from plainbox.impl.commands.cmd_selftest import SelfTestCommand
11
12@@ -49,8 +50,13 @@
13 return self.get_config_cls().get()
14
15 def _load_providers(self):
16+ # Load all normal providers
17 all_providers.load()
18- return all_providers.get_all_plugin_objects()
19+ return [
20+ special.get_manifest(),
21+ # NOTE: currently disabled as it was not added implicitly earlier
22+ # special.get_categories(),
23+ ] + all_providers.get_all_plugin_objects()
24
25 @classmethod
26 def get_exec_version(cls):

Subscribers

People subscribed via source and target branches