Skip to content

Commit

Permalink
bug 1493623 - don't try to collect compiler type for build telemetry …
Browse files Browse the repository at this point in the history
…in artifact builds. r=froydnj

Differential Revision: https://phabricator.services.mozilla.com/D6669

--HG--
extra : moz-landing-system : lando
  • Loading branch information
luser committed Sep 24, 2018
1 parent cd95e89 commit 9a113c8
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions python/mozbuild/mozbuild/telemetry.py
Original file line number Diff line number Diff line change
Expand Up @@ -199,10 +199,9 @@ def get_build_opts(substs):
Translate selected items from `substs` into `build_opts` keys in the schema.
'''
try:
return {
opts = {
k: ty(substs.get(s, None)) for (k, s, ty) in (
# Selected substitutions.
('compiler', 'CC_TYPE', str),
('artifact', 'MOZ_ARTIFACT_BUILDS', bool),
('debug', 'MOZ_DEBUG', bool),
('opt', 'MOZ_OPTIMIZE', bool),
Expand All @@ -211,6 +210,10 @@ def get_build_opts(substs):
# TODO: detect icecream: https://bugzilla.mozilla.org/show_bug.cgi?id=1481614
)
}
compiler = substs.get('CC_TYPE', None)
if compiler:
opts['compiler'] = str(compiler)
return opts
except BuildEnvironmentNotFoundException:
return {}

Expand Down

0 comments on commit 9a113c8

Please sign in to comment.