1 |
commit: d6e7a23bf366eab5c86a3f867bdd7a55325a41fa |
2 |
Author: Matt Turner <mattst88 <AT> gentoo <DOT> org> |
3 |
AuthorDate: Thu Oct 29 00:28:24 2020 +0000 |
4 |
Commit: Matt Turner <mattst88 <AT> gentoo <DOT> org> |
5 |
CommitDate: Thu Oct 29 00:28:24 2020 +0000 |
6 |
URL: https://gitweb.gentoo.org/proj/catalyst.git/commit/?id=d6e7a23b |
7 |
|
8 |
catalyst: Move action_sequence out of self.settings[] |
9 |
|
10 |
This self.settings[] dictionary is very similar to the god object |
11 |
anti-pattern. |
12 |
|
13 |
Signed-off-by: Matt Turner <mattst88 <AT> gentoo.org> |
14 |
|
15 |
catalyst/base/stagebase.py | 17 +++++++++-------- |
16 |
catalyst/targets/embedded.py | 2 +- |
17 |
catalyst/targets/livecd_stage1.py | 2 +- |
18 |
catalyst/targets/livecd_stage2.py | 6 +++--- |
19 |
catalyst/targets/netboot.py | 2 +- |
20 |
catalyst/targets/stage1.py | 12 ++++++------ |
21 |
catalyst/targets/stage4.py | 2 +- |
22 |
7 files changed, 22 insertions(+), 21 deletions(-) |
23 |
|
24 |
diff --git a/catalyst/base/stagebase.py b/catalyst/base/stagebase.py |
25 |
index e71ce344..19c248e3 100644 |
26 |
--- a/catalyst/base/stagebase.py |
27 |
+++ b/catalyst/base/stagebase.py |
28 |
@@ -63,6 +63,7 @@ class StageBase(TargetBase, ClearBase, GenBase): |
29 |
"portage_overlay", |
30 |
"portage_prefix", |
31 |
]) |
32 |
+ self.action_sequence = [] |
33 |
|
34 |
self.set_valid_build_kernel_vars(addlargs) |
35 |
TargetBase.__init__(self, myspec, addlargs) |
36 |
@@ -476,13 +477,13 @@ class StageBase(TargetBase, ClearBase, GenBase): |
37 |
Or it calls the normal set_action_sequence() for the target stage. |
38 |
""" |
39 |
if "purgeonly" in self.settings["options"]: |
40 |
- self.settings["action_sequence"] = ["remove_chroot"] |
41 |
+ self.action_sequence = ["remove_chroot"] |
42 |
return |
43 |
self.set_action_sequence() |
44 |
|
45 |
def set_action_sequence(self): |
46 |
"""Set basic stage1, 2, 3 action sequences""" |
47 |
- self.settings['action_sequence'] = [ |
48 |
+ self.action_sequence = [ |
49 |
"unpack", |
50 |
"setup_confdir", |
51 |
"portage_overlay", |
52 |
@@ -498,14 +499,14 @@ class StageBase(TargetBase, ClearBase, GenBase): |
53 |
|
54 |
def set_completion_action_sequences(self): |
55 |
if "fetch" not in self.settings["options"]: |
56 |
- self.settings["action_sequence"].append("capture") |
57 |
+ self.action_sequence.append("capture") |
58 |
if "keepwork" in self.settings["options"]: |
59 |
- self.settings["action_sequence"].append("clear_autoresume") |
60 |
+ self.action_sequence.append("clear_autoresume") |
61 |
elif "seedcache" in self.settings["options"]: |
62 |
- self.settings["action_sequence"].append("remove_autoresume") |
63 |
+ self.action_sequence.append("remove_autoresume") |
64 |
else: |
65 |
- self.settings["action_sequence"].append("remove_autoresume") |
66 |
- self.settings["action_sequence"].append("remove_chroot") |
67 |
+ self.action_sequence.append("remove_autoresume") |
68 |
+ self.action_sequence.append("remove_chroot") |
69 |
|
70 |
def set_use(self): |
71 |
use = self.settings["spec_prefix"] + "/use" |
72 |
@@ -1367,7 +1368,7 @@ class StageBase(TargetBase, ClearBase, GenBase): |
73 |
self.purge() |
74 |
|
75 |
failure = False |
76 |
- for x in self.settings["action_sequence"]: |
77 |
+ for x in self.action_sequence: |
78 |
log.notice('--- Running action sequence: %s', x) |
79 |
sys.stdout.flush() |
80 |
try: |
81 |
|
82 |
diff --git a/catalyst/targets/embedded.py b/catalyst/targets/embedded.py |
83 |
index 99739512..3899cf1b 100644 |
84 |
--- a/catalyst/targets/embedded.py |
85 |
+++ b/catalyst/targets/embedded.py |
86 |
@@ -41,7 +41,7 @@ class embedded(StageBase): |
87 |
StageBase.__init__(self, spec, addlargs) |
88 |
|
89 |
def set_action_sequence(self): |
90 |
- self.settings['action_sequence'] = [ |
91 |
+ self.action_sequence = [ |
92 |
"unpack", |
93 |
"config_profile_link", |
94 |
"setup_confdir", |
95 |
|
96 |
diff --git a/catalyst/targets/livecd_stage1.py b/catalyst/targets/livecd_stage1.py |
97 |
index f0b6be8b..b8c26cb1 100644 |
98 |
--- a/catalyst/targets/livecd_stage1.py |
99 |
+++ b/catalyst/targets/livecd_stage1.py |
100 |
@@ -23,7 +23,7 @@ class livecd_stage1(StageBase): |
101 |
StageBase.__init__(self, spec, addlargs) |
102 |
|
103 |
def set_action_sequence(self): |
104 |
- self.settings['action_sequence'] = [ |
105 |
+ self.action_sequence = [ |
106 |
"unpack", |
107 |
"config_profile_link", |
108 |
"setup_confdir", |
109 |
|
110 |
diff --git a/catalyst/targets/livecd_stage2.py b/catalyst/targets/livecd_stage2.py |
111 |
index 88c0d95c..cac16b6e 100644 |
112 |
--- a/catalyst/targets/livecd_stage2.py |
113 |
+++ b/catalyst/targets/livecd_stage2.py |
114 |
@@ -87,7 +87,7 @@ class livecd_stage2(StageBase): |
115 |
print_traceback=True) |
116 |
|
117 |
def set_action_sequence(self): |
118 |
- self.settings['action_sequence'] = [ |
119 |
+ self.action_sequence = [ |
120 |
"unpack", |
121 |
"config_profile_link", |
122 |
"setup_confdir", |
123 |
@@ -99,7 +99,7 @@ class livecd_stage2(StageBase): |
124 |
"build_kernel" |
125 |
] |
126 |
if "fetch" not in self.settings["options"]: |
127 |
- self.settings['action_sequence'] += [ |
128 |
+ self.action_sequence += [ |
129 |
"bootloader", |
130 |
"preclean", |
131 |
"livecd_update", |
132 |
@@ -115,4 +115,4 @@ class livecd_stage2(StageBase): |
133 |
"setup_overlay", |
134 |
"create_iso", |
135 |
] |
136 |
- self.settings["action_sequence"].append("clear_autoresume") |
137 |
+ self.action_sequence.append("clear_autoresume") |
138 |
|
139 |
diff --git a/catalyst/targets/netboot.py b/catalyst/targets/netboot.py |
140 |
index 5620e0d3..61583f0d 100644 |
141 |
--- a/catalyst/targets/netboot.py |
142 |
+++ b/catalyst/targets/netboot.py |
143 |
@@ -160,7 +160,7 @@ class netboot(StageBase): |
144 |
self.resume.enable("empty") |
145 |
|
146 |
def set_action_sequence(self): |
147 |
- self.settings['action_sequence'] = [ |
148 |
+ self.action_sequence = [ |
149 |
"unpack", |
150 |
"config_profile_link", |
151 |
"setup_confdir", |
152 |
|
153 |
diff --git a/catalyst/targets/stage1.py b/catalyst/targets/stage1.py |
154 |
index 2c09a41f..89b30fe1 100644 |
155 |
--- a/catalyst/targets/stage1.py |
156 |
+++ b/catalyst/targets/stage1.py |
157 |
@@ -87,15 +87,15 @@ class stage1(StageBase): |
158 |
chroot for re-use in stage2 without the need to unpack it. |
159 |
''' |
160 |
if "fetch" not in self.settings["options"]: |
161 |
- self.settings["action_sequence"].append("capture") |
162 |
+ self.action_sequence.append("capture") |
163 |
if "keepwork" in self.settings["options"]: |
164 |
- self.settings["action_sequence"].append("clear_autoresume") |
165 |
+ self.action_sequence.append("clear_autoresume") |
166 |
elif "seedcache" in self.settings["options"]: |
167 |
- self.settings["action_sequence"].append("remove_autoresume") |
168 |
- self.settings["action_sequence"].append("clean_stage1") |
169 |
+ self.action_sequence.append("remove_autoresume") |
170 |
+ self.action_sequence.append("clean_stage1") |
171 |
else: |
172 |
- self.settings["action_sequence"].append("remove_autoresume") |
173 |
- self.settings["action_sequence"].append("remove_chroot") |
174 |
+ self.action_sequence.append("remove_autoresume") |
175 |
+ self.action_sequence.append("remove_chroot") |
176 |
|
177 |
def clean_stage1(self): |
178 |
'''seedcache is enabled, so salvage the /tmp/stage1root, |
179 |
|
180 |
diff --git a/catalyst/targets/stage4.py b/catalyst/targets/stage4.py |
181 |
index 346c0845..eef24a73 100644 |
182 |
--- a/catalyst/targets/stage4.py |
183 |
+++ b/catalyst/targets/stage4.py |
184 |
@@ -39,7 +39,7 @@ class stage4(StageBase): |
185 |
self.settings["cleanables"].remove('/etc/resolv.conf') |
186 |
|
187 |
def set_action_sequence(self): |
188 |
- self.settings['action_sequence'] = [ |
189 |
+ self.action_sequence = [ |
190 |
"unpack", |
191 |
"config_profile_link", |
192 |
"setup_confdir", |