[PATCHv6,1/3] configure.ac: disable shared library for non abi compat mode

Message ID 1480003389-19250-2-git-send-email-maxim.uvarov@linaro.org
State New
Headers show

Commit Message

Maxim Uvarov Nov. 24, 2016, 4:03 p.m.
original configure.ac enables abi compat mode by default,
even without --enable-abi-compat provided. And has broken
logic to disable abi compat mode. Correct logic to build abi
compat mode and option to disable it. Shared library is not
needed for non abi compat mode, so turn it off.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

---
 configure.ac | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

-- 
2.7.1.250.gff4ea60

Patch

diff --git a/configure.ac b/configure.ac
index be5a292..7cb0c7a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -241,16 +241,19 @@  ODP_CFLAGS="$ODP_CFLAGS -DODP_DEBUG=$ODP_DEBUG"
 ODP_ABI_COMPAT=1
 abi_compat=yes
 AC_ARG_ENABLE([abi-compat],
-    [  --enable-abi-compat     build all targets in ABI compatible mode (default=yes)],
+    [  --disable-abi-compat    disable build all targets in ABI compatible mode (default=no)],
     [if test "x$enableval" = "xyes"; then
 	ODP_ABI_COMPAT=1
-	abi_compat=yes
      else
 	ODP_ABI_COMPAT=0
-	abi_compat=no
     fi])
 AC_SUBST(ODP_ABI_COMPAT)
 
+if test $ODP_ABI_COMPAT -eq 0; then
+	enable_shared=no
+	abi_compat=no
+fi
+
 ##########################################################################
 # Default warning setup
 ##########################################################################