diff options
| author | Michael Stahl <mstahl@redhat.com> | 2012-09-06 18:43:15 +0200 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2012-09-06 18:46:41 +0200 |
| commit | d9cbc837f84c415f5949a6893764dae8268f0d85 (patch) | |
| tree | c3afc172a3b2a7059578441bcff0365beeaf1fc5 /solenv | |
| parent | 47919e05f68a6871031b92ad028e4345a51bf5c9 (diff) | |
gbuild: disable symbols on --enable-dbgutil --disable-symbols
Due to the setup of gb_DEBUGLEVEL in gbuild.mk, gb_SYMBOL was always
enabled when --enable-dbgutil is set, with no way to override it.
Fix that by turning configure's ENABLE_SYMBOLS into a tri-state, where
the new "FALSE" value, set by an explicit --disable-symbols, overrides
any implicit way of enabling symbols.
But by default an --enable-dbgutil still enables gb_SYMBOL.
Change-Id: I94c609863980ed1ab9c73d7a4861c394442b531d
Diffstat (limited to 'solenv')
| -rw-r--r-- | solenv/gbuild/gbuild.mk | 16 | ||||
| -rw-r--r-- | solenv/inc/settings.mk | 2 |
2 files changed, 11 insertions, 7 deletions
diff --git a/solenv/gbuild/gbuild.mk b/solenv/gbuild/gbuild.mk index e1be999bbeea..d9da37d74a40 100644 --- a/solenv/gbuild/gbuild.mk +++ b/solenv/gbuild/gbuild.mk @@ -76,12 +76,6 @@ else gb_PRODUCT := $(false) endif -ifneq ($(strip $(ENABLE_SYMBOLS)$(enable_symbols)),) -gb_SYMBOL := $(true) -else -gb_SYMBOL := $(false) -endif - gb_TIMELOG := 0 ifneq ($(strip $(TIMELOG)$(timelog)),) gb_TIMELOG := 1 @@ -117,8 +111,18 @@ ENABLE_DEBUG_FOR := all endif endif +ifeq ($(or $(ENABLE_SYMBOLS),$(enable_symbols)),FALSE) +gb_SYMBOL := $(false) +else +ifneq ($(strip $(ENABLE_SYMBOLS)$(enable_symbols)),) +gb_SYMBOL := $(true) +else ifneq ($(gb_DEBUGLEVEL),0) gb_SYMBOL := $(true) +else +gb_SYMBOL := $(false) +endif +endif endif ifneq ($(nodep),) diff --git a/solenv/inc/settings.mk b/solenv/inc/settings.mk index 230c06094510..e24c0e6cd1d3 100644 --- a/solenv/inc/settings.mk +++ b/solenv/inc/settings.mk @@ -1040,7 +1040,7 @@ CDEFS+= -DGXX_INCLUDE_PATH=$(GXX_INCLUDE_PATH) CDEFS+= -DSUPD=$(UPD) # flags to enable build with symbols; required for crashdump feature -.IF ("$(ENABLE_CRASHDUMP)"!="" && "$(ENABLE_CRASHDUMP)"!="DUMMY") || "$(ENABLE_SYMBOLS)"!="" +.IF ("$(ENABLE_CRASHDUMP)"!="" && "$(ENABLE_CRASHDUMP)"!="DUMMY") || ("$(ENABLE_SYMBOLS)"!="" && "$(ENABLE_SYMBOLS)"!="FALSE") # if debug is enabled, this may enable less debug info than debug, so rely just on debug .IF "$(debug)" == "" CFLAGSENABLESYMBOLS_CC_ONLY*=$(CFLAGSENABLESYMBOLS) |