Skip to content

Commit 8541332

Browse files
committed
use options.debug for branching
1 parent ed19665 commit 8541332

1 file changed

Lines changed: 5 additions & 24 deletions

File tree

configure.py

Lines changed: 5 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1627,17 +1627,6 @@ def set_configuration_variable(configs, name, release=None, debug=None):
16271627
configs['Release']['variables'][name] = release
16281628
configs['Debug']['variables'][name] = debug
16291629

1630-
def set_configuration_variable_and_defines(configs, name, release=None, debug=None, release_define=None, debug_define=None):
1631-
set_configuration_variable(configs, name, release, debug)
1632-
if configs['Debug'].get('defines') is None:
1633-
configs['Debug']['defines'] = []
1634-
if configs['Release'].get('defines') is None:
1635-
configs['Release']['defines'] = []
1636-
if debug_define:
1637-
configs['Debug']['defines'].append(debug_define)
1638-
if release_define:
1639-
configs['Release']['defines'].append(release_define)
1640-
16411630
def configure_arm(o):
16421631
if options.arm_float_abi:
16431632
arm_float_abi = options.arm_float_abi
@@ -1980,14 +1969,7 @@ def configure_library(lib, output, pkgname=None):
19801969

19811970

19821971
def configure_v8(o, configs):
1983-
set_configuration_variable_and_defines(
1984-
configs,
1985-
'v8_enable_v8_checks',
1986-
release='0',
1987-
debug='1',
1988-
release_define=None,
1989-
debug_define='V8_ENABLE_CHECKS',
1990-
)
1972+
set_configuration_variable(configs, 'v8_enable_v8_checks', release=0, debug=1)
19911973

19921974
o['variables']['v8_enable_webassembly'] = 0 if options.v8_lite_mode else 1
19931975
o['variables']['v8_enable_javascript_promise_hooks'] = 1
@@ -2618,11 +2600,10 @@ def make_bin_override():
26182600
del configurations['Release']['variables']
26192601
config_debug_vars = configurations['Debug']['variables']
26202602
del configurations['Debug']['variables']
2621-
output['conditions'].append(['build_type=="Release"', {
2622-
'variables': config_release_vars,
2623-
}, {
2624-
'variables': config_debug_vars,
2625-
}])
2603+
if options.debug:
2604+
variables = variables | config_debug_vars
2605+
else:
2606+
variables = variables | config_release_vars
26262607

26272608
# make_global_settings should be a root level element too
26282609
if 'make_global_settings' in output:

0 commit comments

Comments
 (0)