Bug 1442378 - part 5 - make multiple occurrences of the same file fatal; r=jmaher

We shouldn't really have this problem, and shouting about it early will
make sure that we address issues like multiple instances of omni.ja,
rather than not recording information that we should have been.
This commit is contained in:
Nathan Froyd 2018-03-02 08:34:39 -05:00
Родитель dcc7118281
Коммит 6bd724561d
1 изменённых файлов: 7 добавлений и 10 удалений

Просмотреть файл

@ -1612,17 +1612,14 @@ or run without that action (ie: --no-{action})"
if containing_dir == 'browser':
name = 'browser-omni.ja'
if name in subtests:
# File seen twice in same archive;
# ignore to avoid confusion.
subtests[name] = None
else:
subtests[name] = size
self.fatal('should not see %s (%s) multiple times!'
% (name, path))
subtests[name] = size
for name in subtests:
if subtests[name] is not None:
self.info('Size of %s: %s bytes' % (name,
subtests[name]))
size_measurements.append(
{'name': name, 'value': subtests[name]})
self.info('Size of %s: %s bytes' % (name,
subtests[name]))
size_measurements.append(
{'name': name, 'value': subtests[name]})
except:
self.info('Unable to search %s for component sizes.' % installer)
size_measurements = []