]> Savannah Git Hosting - gnulib.git/commitdiff
gnulib-tool.py: Follow gnulib-tool changes, part 16.
authorBruno Haible <bruno@clisp.org>
Fri, 29 Jul 2022 20:58:27 +0000 (22:58 +0200)
committerBruno Haible <bruno@clisp.org>
Fri, 29 Jul 2022 20:58:27 +0000 (22:58 +0200)
Follow gnulib-tool change
2015-09-25  Pavel Raiskup  <praiskup@redhat.com>
gnulib-common.m4: fix gl_PROG_AR_RANLIB/AM_PROG_AR clash

* pygnulib/GLImport.py (GLImport.gnulib_comp): Put the
gl_USE_SYSTEM_EXTENSIONS right before gl_PROG_AR_RANLIB into
gnulib-comp.m4 (if the 'extensions' module is used).

ChangeLog
gnulib-tool.py.TODO
pygnulib/GLImport.py

index 01d8df2e13a3f971f8e1e97ced5135596956056e..1ddf2822ade3f8f0428da2918706a2bed50b8d29 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,13 @@
 2022-07-29  Bruno Haible  <bruno@clisp.org>
 
+       gnulib-tool.py: Follow gnulib-tool changes, part 16.
+       Follow gnulib-tool change
+       2015-09-25  Pavel Raiskup  <praiskup@redhat.com>
+       gnulib-common.m4: fix gl_PROG_AR_RANLIB/AM_PROG_AR clash
+       * pygnulib/GLImport.py (GLImport.gnulib_comp): Put the
+       gl_USE_SYSTEM_EXTENSIONS right before gl_PROG_AR_RANLIB into
+       gnulib-comp.m4 (if the 'extensions' module is used).
+
        gnulib-tool.py: Modernize coding style.
        * pygnulib/*.py: Remove parentheses around return value expressions.
 
index 7017d15a052488ca335519074b526dfd9de65a56..bcb811b5dfe73392c77827cf1aec0b73c358c732 100644 (file)
@@ -1134,34 +1134,6 @@ Date:   Tue Oct 6 13:20:05 2015 +0200
     (func_import, func_create_testdir): All dumps of gl_PROG_AR_RANLIB
     replaced with func_emit_pre_early_macros call.
 
-commit f8fe25fab60e3c687a12446984bb475342956bb8
-Author: Pavel Raiskup <praiskup@redhat.com>
-Date:   Fri Sep 25 11:25:03 2015 -0700
-
-    gnulib-common.m4: fix gl_PROG_AR_RANLIB/AM_PROG_AR clash
-
-    The gl_PROG_AR_RANLIB (it is always called by gl_EARLY) sets AR
-    and ARFLAGS variables.  Doing this unconditionally could break
-    later Automake's AM_PROG_AR invocation (at least it's
-    AC_CHECK_TOOLS call to detect correct 'ar' binary).
-
-    Original purpose of the gl_PROG_AR_RANLIB was only to handle the
-    Amsterdam Compiler Kit, so make the previous code to have effects
-    only on ACK, and rather automatically call the Automake's
-    AM_PROG_AR as soon as possible to decide other cases.
-
-    References:
-    http://lists.gnu.org/archive/html/bug-gnulib/2015-07/msg00001.html
-
-    * m4/gnulib-common.m4 (gl_PROG_AR_RANLIB): AC_BEFORE AM_PROG_AR.
-    Set the AR/ARFLAGS to ACK defaults OR call AM_PROG_AR.  If neither
-    is possible, keep setting AR/ARFLAGS to reasonable defaults.
-    * gnulib-tool (func_import): Put the gl_USE_SYSTEM_EXTENSIONS
-    right before gl_PROG_AR_RANLIB into gnulib-comp.m4 (if the
-    'extensions' module is used.
-    * modules/extensions (configure.ac-early): Remove as this snippet
-    is added to gnulib-comp.m4 earlier anyway.
-
 commit 9bdf6c8a0cdeb13c12e4b65dee9538c5468dbe1d
 Author: Bruno Haible <bruno@clisp.org>
 Date:   Sun Aug 19 14:06:50 2012 +0200
index 10ad069271ed3b0af4d56c35edd12be1e4f62a80..c51b5036df3563cec28781fe8a00a4cb77b20de9 100644 (file)
@@ -598,8 +598,10 @@ AC_DEFUN([%s_EARLY],
   m4_pattern_forbid([^gl_[A-Z]])dnl the gnulib macro namespace
   m4_pattern_allow([^gl_ES$])dnl a valid locale name
   m4_pattern_allow([^gl_LIBOBJS$])dnl a variable
-  m4_pattern_allow([^gl_LTLIBOBJS$])dnl a variable
-  AC_REQUIRE([gl_PROG_AR_RANLIB])\n''' % (configure_ac, macro_prefix)
+  m4_pattern_allow([^gl_LTLIBOBJS$])dnl a variable\n''' % (configure_ac, macro_prefix)
+        if any(str(module) == 'extensions' for module in moduletable['final']):
+            emit += '  AC_REQUIRE([gl_USE_SYSTEM_EXTENSIONS])\n'
+        emit += '  AC_REQUIRE([gl_PROG_AR_RANLIB])\n'
         uses_subdirs = False
         for module in moduletable['main']:
             # Test whether there are some source files in subdirectories.