forked from mirrors/gecko-dev
Bug 1654994 - Make full symbols archive opt-in rather than opt-out. r=froydnj
Differential Revision: https://phabricator.services.mozilla.com/D85354
This commit is contained in:
parent
6f9d5521fd
commit
13b57c569e
3 changed files with 7 additions and 7 deletions
10
Makefile.in
10
Makefile.in
|
|
@ -211,13 +211,13 @@ symbolsarchive: prepsymbolsarchive
|
||||||
$(abspath $(DIST)/crashreporter-symbols))
|
$(abspath $(DIST)/crashreporter-symbols))
|
||||||
|
|
||||||
ifdef MOZ_CRASHREPORTER
|
ifdef MOZ_CRASHREPORTER
|
||||||
# Set MOZ_DISABLE_FULL_SYMBOLS to disable generation and upload of the full
|
# Set MOZ_ENABLE_FULL_SYMBOLS to enable generation and upload of the full
|
||||||
# crashreporter symbols archives
|
# crashreporter symbols archives
|
||||||
ifdef MOZ_DISABLE_FULL_SYMBOLS
|
ifdef MOZ_ENABLE_FULL_SYMBOLS
|
||||||
buildsymbols: symbolsarchive
|
|
||||||
else
|
|
||||||
buildsymbols: symbolsfullarchive symbolsarchive
|
buildsymbols: symbolsfullarchive symbolsarchive
|
||||||
endif # MOZ_DISABLE_FULL_SYMBOLS
|
else
|
||||||
|
buildsymbols: symbolsarchive
|
||||||
|
endif # MOZ_ENABLE_FULL_SYMBOLS
|
||||||
else
|
else
|
||||||
buildsymbols:
|
buildsymbols:
|
||||||
endif
|
endif
|
||||||
|
|
|
||||||
|
|
@ -2,7 +2,7 @@
|
||||||
|
|
||||||
# Enable the artifact build.
|
# Enable the artifact build.
|
||||||
ac_add_options --enable-artifact-builds
|
ac_add_options --enable-artifact-builds
|
||||||
if test -n "$MOZ_ARTIFACT_TASK_WIN32_OPT" -a -z "$MOZ_DISABLE_FULL_SYMBOLS"; then
|
if test -n "$MOZ_ARTIFACT_TASK_WIN32_OPT" -a -n "$MOZ_ENABLE_FULL_SYMBOLS"; then
|
||||||
ac_add_options --enable-artifact-build-symbols=full
|
ac_add_options --enable-artifact-build-symbols=full
|
||||||
else
|
else
|
||||||
ac_add_options --enable-artifact-build-symbols
|
ac_add_options --enable-artifact-build-symbols
|
||||||
|
|
|
||||||
|
|
@ -188,8 +188,8 @@ def enable_full_crashsymbols(config, jobs):
|
||||||
enable_full_crashsymbols = job['attributes'].get('enable-full-crashsymbols')
|
enable_full_crashsymbols = job['attributes'].get('enable-full-crashsymbols')
|
||||||
if enable_full_crashsymbols and config.params['project'] in branches:
|
if enable_full_crashsymbols and config.params['project'] in branches:
|
||||||
logger.debug("Enabling full symbol generation for %s", job['name'])
|
logger.debug("Enabling full symbol generation for %s", job['name'])
|
||||||
|
job['worker']['env']['MOZ_ENABLE_FULL_SYMBOLS'] = '1'
|
||||||
else:
|
else:
|
||||||
logger.debug("Disabling full symbol generation for %s", job['name'])
|
logger.debug("Disabling full symbol generation for %s", job['name'])
|
||||||
job['worker']['env']['MOZ_DISABLE_FULL_SYMBOLS'] = '1'
|
|
||||||
job['attributes'].pop('enable-full-crashsymbols', None)
|
job['attributes'].pop('enable-full-crashsymbols', None)
|
||||||
yield job
|
yield job
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue