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.' % \ |
