diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-07-08 17:49:51 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-07-11 08:57:44 +0000 |
commit | 436ee0989d1bc321886fac5a3507fc1d16650abd (patch) | |
tree | 03509d40005d7a21559a4f578fe0082e5df56827 /configure.ac | |
parent | Break gb_DEBUGINFO_FLAGS out of gb_DEBUG_CFLAGS (diff) | |
download | core-436ee0989d1bc321886fac5a3507fc1d16650abd.tar.gz core-436ee0989d1bc321886fac5a3507fc1d16650abd.zip |
Get rid of odd ENABLE_SYMBOLS=FALSE
* None of the other ENABLE_* support anything but TRUE or empty.
* Other checks for -n "$ENABLE_SYMBOLS" in configure.ac are confused by it.
* solenv/gbuild/gbuild.help.txt talks about "If not empty", not mentioning "FALSE".
* A missing (redundant) explicit --disable-symbols left gb_SYMBOL uninitialized
(instead of setting it to $(false))---but without consequences, as all the places
checking gb_SYMBOL explicitly check for $(true).
Change-Id: Id004189de27a7936862dab3a5fd84d549c06af2b
Reviewed-on: https://gerrit.libreoffice.org/27057
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'configure.ac')
-rw-r--r-- | configure.ac | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/configure.ac b/configure.ac index f5f9da655823..11bb4680446c 100644 --- a/configure.ac +++ b/configure.ac @@ -4044,11 +4044,7 @@ if test -n "$enable_symbols" -a "$enable_symbols" != "no"; then fi AC_MSG_RESULT([yes]) else - if test -n "$enable_symbols" -a "$enable_symbols" = "no"; then - ENABLE_SYMBOLS="FALSE" - else - ENABLE_SYMBOLS= - fi + ENABLE_SYMBOLS= AC_MSG_RESULT([no]) fi AC_SUBST(ENABLE_SYMBOLS) |