Don't build readline/libreadline.a, when --with-system-readline is supplied
authorДилян Палаузов <dilyan.palauzov@aegee.org>
Wed, 12 Dec 2018 07:46:03 +0000 (07:46 +0000)
committerSimon Marchi <simon.marchi@polymtl.ca>
Thu, 3 Jan 2019 05:03:34 +0000 (00:03 -0500)
https://sourceware.org/bugzilla/show_bug.cgi?id=18632

The bundled libreadline is always built, even if the system is
./configure'd --with-system-readline and the build libreadline.a is not
used.

Proposed patch:

Fix ./configure.ac not to proceed readline/, when --with-system-
readline is provided

ChangeLog
configure
configure.ac

index 60b19fdb150f829754f1c4f7c5d90a83cd7efec8..81654d0c70df88b23e7d49bf9e798d1819243ec3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2019-01-03  Дилян Палаузов  <dilyan.palauzov@aegee.org>
+
+       * configure.ac: Don't configure readline if --with-system-readline is
+       used.
+       * configure: Re-generate.
+
 2018-10-31  Joseph Myers  <joseph@codesourcery.com>
 
        Merge from GCC:
index 08dd35c3eaf553ad958db6b4f3fac1efa03cfb80..374764596123660de16656a0f5d4524a3b9c9ee7 100755 (executable)
--- a/configure
+++ b/configure
@@ -2907,6 +2907,12 @@ if test x$with_system_zlib = xyes ; then
   noconfigdirs="$noconfigdirs zlib"
 fi
 
+# Don't compile the bundled readline/libreadline.a if --with-system-readline
+# is provided.
+if test x$with_system_readline = xyes ; then
+  noconfigdirs="$noconfigdirs readline"
+fi
+
 # some tools are so dependent upon X11 that if we're not building with X,
 # it's not even worth trying to configure, much less build, that tool.
 
index 99229d97037a9ece6d6003c7833a49539590ea14..46501c28826985e2fed1509c93e7e4c9093b88eb 100644 (file)
@@ -245,6 +245,12 @@ if test x$with_system_zlib = xyes ; then
   noconfigdirs="$noconfigdirs zlib"
 fi
 
+# Don't compile the bundled readline/libreadline.a if --with-system-readline
+# is provided.
+if test x$with_system_readline = xyes ; then
+  noconfigdirs="$noconfigdirs readline"
+fi
+
 # some tools are so dependent upon X11 that if we're not building with X, 
 # it's not even worth trying to configure, much less build, that tool.