Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Revert "Bug 967516 - Enable DEVICE_DEBUG by default for non productio…
Browse files Browse the repository at this point in the history
…n builds r=ochameau"

This reverts commit 2cf5528.
  • Loading branch information
KevinGrandon committed Oct 1, 2014
1 parent 3ff469b commit 1b0c386
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 15 deletions.
23 changes: 10 additions & 13 deletions Makefile
Expand Up @@ -114,6 +114,16 @@ REMOTE_DEBUGGER?=0
# Debug mode for build process
BUILD_DEBUG?=0

ifeq ($(DEVICE_DEBUG),1)
REMOTE_DEBUGGER=1
NO_LOCK_SCREEN=1
SCREEN_TIMEOUT=300
endif

ifeq ($(SIMULATOR),1)
SCREEN_TIMEOUT=0
endif

# We also disable FTU when running in Firefox or in debug mode
ifeq ($(DEBUG),1)
NOFTU=1
Expand Down Expand Up @@ -185,19 +195,6 @@ endif
ifeq ($(PRODUCTION), 1)
GAIA_OPTIMIZE=1
GAIA_APP_TARGET=production
else ifneq ($(DEBUG),1)
# Enable DEVICE_DEBUG for non-production, non-debug builds.
DEVICE_DEBUG=1
endif

ifeq ($(DEVICE_DEBUG),1)
REMOTE_DEBUGGER=1
SCREEN_TIMEOUT=300
endif

ifeq ($(SIMULATOR),1)
SCREEN_TIMEOUT=0
NO_LOCK_SCREEN=1
endif

ifeq ($(DOGFOOD), 1)
Expand Down
3 changes: 1 addition & 2 deletions build/test/integration/build.test.js
Expand Up @@ -363,8 +363,7 @@ suite('Build Integration tests', function() {
var ignoreSettings = [
'apz.force-enable',
'debug.console.enabled',
'developer.menu.enabled',
'screen.timeout'
'developer.menu.enabled'
];
ignoreSettings.forEach(function(key) {
if (commonSettings[key] !== undefined) {
Expand Down

0 comments on commit 1b0c386

Please sign in to comment.