diff options
Diffstat (limited to 'scripts/lib/checklayer')
-rw-r--r-- | scripts/lib/checklayer/__init__.py | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/scripts/lib/checklayer/__init__.py b/scripts/lib/checklayer/__init__.py index e69a10f452..f91888ccbb 100644 --- a/scripts/lib/checklayer/__init__.py +++ b/scripts/lib/checklayer/__init__.py | |||
@@ -156,6 +156,27 @@ def _find_layer(depend, layers): | |||
156 | return layer | 156 | return layer |
157 | return None | 157 | return None |
158 | 158 | ||
159 | def sanity_check_layers(layers, logger): | ||
160 | """ | ||
161 | Check that we didn't find duplicate collection names, as the layer that will | ||
162 | be used is non-deterministic. The precise check is duplicate collections | ||
163 | with different patterns, as the same pattern being repeated won't cause | ||
164 | problems. | ||
165 | """ | ||
166 | import collections | ||
167 | |||
168 | passed = True | ||
169 | seen = collections.defaultdict(set) | ||
170 | for layer in layers: | ||
171 | for name, data in layer.get("collections", {}).items(): | ||
172 | seen[name].add(data["pattern"]) | ||
173 | |||
174 | for name, patterns in seen.items(): | ||
175 | if len(patterns) > 1: | ||
176 | passed = False | ||
177 | logger.error("Collection %s found multiple times: %s" % (name, ", ".join(patterns))) | ||
178 | return passed | ||
179 | |||
159 | def get_layer_dependencies(layer, layers, logger): | 180 | def get_layer_dependencies(layer, layers, logger): |
160 | def recurse_dependencies(depends, layer, layers, logger, ret = []): | 181 | def recurse_dependencies(depends, layer, layers, logger, ret = []): |
161 | logger.debug('Processing dependencies %s for layer %s.' % \ | 182 | logger.debug('Processing dependencies %s for layer %s.' % \ |