[COMMITTED] Don't build readline's shared libs by default

Message ID 1437838713-8216-1-git-send-email-patrick@parcs.ath.cx
State New, archived
Headers

Commit Message

Patrick Palka July 25, 2015, 3:38 p.m. UTC
  Since the sync to version 7.0-alpho, readline now by default builds
(unused) shared libraries alongside static libraries, whereas before it
only built static libraries.  A couple of GDB buildbots were not happy
with this change:

  http://gdb-build.sergiodj.net/builders/AIX-POWER7-plain/builds/240
  http://gdb-build.sergiodj.net/builders/Fedora-i686/builds/1518

To get these buildbots building again, this patch alters readline's
configure.ac file to not build shared libraries by default, as was the
case with readline 6.2.  A more permanent fix may be to alter the
top-level Makefile.def to pass --disable-shared to readline, or to
investigate why these building these shared libraries are giving the
buildbots trouble.  (I think the proximate reason why the i686 buildbot
fails is because it passes CFLAGS=-m32 instead of CC="gcc -m32" to the
top-level configure script, and readline's linker commands don't inherit
CFLAGS.  Not sure about the AIX failure.)

readline/ChangeLog.gdb:

	* configure.ac: Default opt_shared_libs to no.
	* configure: Regenerate.
---
 readline/ChangeLog.gdb | 5 +++++
 readline/configure     | 2 +-
 readline/configure.ac  | 2 +-
 3 files changed, 7 insertions(+), 2 deletions(-)
  

Comments

Patrick Palka July 25, 2015, 8:02 p.m. UTC | #1
On Sat, Jul 25, 2015 at 11:38 AM, Patrick Palka <patrick@parcs.ath.cx> wrote:
> Since the sync to version 7.0-alpho, readline now by default builds
> (unused) shared libraries alongside static libraries, whereas before it
> only built static libraries.  A couple of GDB buildbots were not happy
> with this change:
>
>   http://gdb-build.sergiodj.net/builders/AIX-POWER7-plain/builds/240
>   http://gdb-build.sergiodj.net/builders/Fedora-i686/builds/1518
>
> To get these buildbots building again, this patch alters readline's
> configure.ac file to not build shared libraries by default, as was the
> case with readline 6.2.  A more permanent fix may be to alter the
> top-level Makefile.def to pass --disable-shared to readline, or to
> investigate why these building these shared libraries are giving the
> buildbots trouble.  (I think the proximate reason why the i686 buildbot
> fails is because it passes CFLAGS=-m32 instead of CC="gcc -m32" to the
> top-level configure script, and readline's linker commands don't inherit
> CFLAGS.  Not sure about the AIX failure.)
>
> readline/ChangeLog.gdb:
>
>         * configure.ac: Default opt_shared_libs to no.
>         * configure: Regenerate.

Reverted, along with the original import commit.  This commit message
is mostly bogus; during the import I failed to notice that
configure.in had a number of local changes of its own, including this
one to disable building shared libraries.  It was not a new change
needed specifically for readline 7.0.
  

Patch

diff --git a/readline/ChangeLog.gdb b/readline/ChangeLog.gdb
index 2a9a1b2..013727d 100644
--- a/readline/ChangeLog.gdb
+++ b/readline/ChangeLog.gdb
@@ -1,5 +1,10 @@ 
 2015-07-25  Patrick Palka  <patrick@parcs.ath.cx>
 
+	* configure.ac: Default opt_shared_libs to no.
+	* configure: Regenerate.
+
+2015-07-25  Patrick Palka  <patrick@parcs.ath.cx>
+
 	Import readline 7.0 alpha
 	* configure: Regenerate.
 	* examples/rlfe/configure: Regenerate.
diff --git a/readline/configure b/readline/configure
index 3af8fbe..813e379 100755
--- a/readline/configure
+++ b/readline/configure
@@ -2352,7 +2352,7 @@  fi
 
 opt_multibyte=yes
 opt_static_libs=yes
-opt_shared_libs=yes
+opt_shared_libs=no
 
 # Check whether --enable-multibyte was given.
 if test "${enable_multibyte+set}" = set; then :
diff --git a/readline/configure.ac b/readline/configure.ac
index 42e21fa..b4fa645 100644
--- a/readline/configure.ac
+++ b/readline/configure.ac
@@ -58,7 +58,7 @@  fi
 dnl option parsing for optional features
 opt_multibyte=yes
 opt_static_libs=yes
-opt_shared_libs=yes
+opt_shared_libs=no
 
 AC_ARG_ENABLE(multibyte, AC_HELP_STRING([--enable-multibyte], [enable multibyte characters if OS supports them]), opt_multibyte=$enableval)
 AC_ARG_ENABLE(shared, AC_HELP_STRING([--enable-shared], [build shared libraries [[default=YES]]]), opt_shared_libs=$enableval)