Bug 1149164 - Fix minor logic error in previous commit;r=gbrown
authorWilliam Lachance <wlachance@mozilla.com>
Mon, 02 Nov 2015 13:36:37 -0500
changeset 270859 1c3782aef6e34f972dd5699329e142a0ddba287a
parent 270858 f8b520ee98535e9bfbe4a5dae90fc108e5628f6e
child 270860 bc13e689699b1957464db27b7e311b26f0856aef
push id29626
push usercbook@mozilla.com
push dateTue, 03 Nov 2015 11:00:46 +0000
treeherdermozilla-central@bb4d614a0b09 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgbrown
bugs1149164
milestone45.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1149164 - Fix minor logic error in previous commit;r=gbrown
testing/mozharness/mozharness/mozilla/building/buildbase.py
--- a/testing/mozharness/mozharness/mozilla/building/buildbase.py
+++ b/testing/mozharness/mozharness/mozilla/building/buildbase.py
@@ -1619,17 +1619,17 @@ or run without that action (ie: --no-{ac
                     name, filesize))
                 if any(name.endswith(extension) for extension in ['apk',
                                                                   'dmg',
                                                                   'bz2',
                                                                   'zip']):
                     installer_size = filesize
                 else:
                     size_measurements.append({'name': name, 'value': filesize})
-        if filesize or size_measurements:
+        if installer_size or size_measurements:
             self.info('PERFHERDER_DATA: %s' % (json.dumps({
                 "framework": {"name": "build_metrics"},
                 "suites": [{"name": "installer size",
                             "value": installer_size,
                             "subtests": size_measurements}]
             })))
 
     def _set_file_properties(self, file_name, find_dir, prop_type,