getopt: Move AC_LIBOBJ invocations to module description.

* m4/getopt.m4 (gl_REPLACE_GETOPT): Remove macro.
(gl_FUNC_GETOPT): Inline it here. Move AC_LIBOBJ and gl_PREREQ_GETOPT
invocations from here...
* modules/getopt-gnu (configure.ac): ... to here.
* modules/getopt-posix (configure.ac): ... and here.
(Depends-on): Update condition.

diff --git a/ChangeLog b/ChangeLog
index 5db6ffe3ed6c43057fff170357cc760010f53dd6..b1c66fd1707dca35b7c36f027f23f01d7d220671 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2011-05-08  Bruno Haible  <bruno@clisp.org>
+
+       getopt: Move AC_LIBOBJ invocations to module description.
+       * m4/getopt.m4 (gl_REPLACE_GETOPT): Remove macro.
+       (gl_FUNC_GETOPT): Inline it here. Move AC_LIBOBJ and gl_PREREQ_GETOPT
+       invocations from here...
+       * modules/getopt-gnu (configure.ac): ... to here.
+       * modules/getopt-posix (configure.ac): ... and here.
+       (Depends-on): Update condition.
+
 2011-05-08  Bruno Haible  <bruno@clisp.org>
 
        getopt, argp: Respect rules for use of AC_LIBOBJ.

diff --git a/m4/getopt.m4 b/m4/getopt.m4
index 4ad52fd54b9f766124530573c2c21e5057ff40ff..4d8450ff907211f50845f552481165937e68e7a2 100644 (file)
--- a/m4/getopt.m4
+++ b/m4/getopt.m4
@@ -23,7 +23,10 @@ AC_DEFUN([gl_FUNC_GETOPT_POSIX],
     [])
   ])
   if test $REPLACE_GETOPT = 1; then
-    gl_REPLACE_GETOPT
+    dnl Arrange for getopt.h to be created.
+    gl_GETOPT_SUBSTITUTE_HEADER
+    dnl Arrange for unistd.h to include getopt.h.
+    GNULIB_UNISTD_H_GETOPT=1
   fi
 ])
 
@@ -37,19 +40,6 @@ AC_DEFUN([gl_FUNC_GETOPT_GNU],
   AC_REQUIRE([gl_FUNC_GETOPT_POSIX])
 ])
 
-# Request the gnulib implementation of the getopt functions unconditionally.
-AC_DEFUN([gl_REPLACE_GETOPT],
-[
-  dnl Arrange for getopt.h to be created.
-  gl_GETOPT_SUBSTITUTE_HEADER
-  dnl Arrange for unistd.h to include getopt.h.
-  GNULIB_UNISTD_H_GETOPT=1
-  dnl Arrange to compile the getopt implementation.
-  AC_LIBOBJ([getopt])
-  AC_LIBOBJ([getopt1])
-  gl_PREREQ_GETOPT
-])
-
 # emacs' configure.in uses this.
 AC_DEFUN([gl_GETOPT_IFELSE],
 [

diff --git a/modules/getopt-gnu b/modules/getopt-gnu
index 92b37ddbed62d8caf184751d79c343177795b917..3e5876a1b94e8ba4704d4367248013cba81bb880 100644 (file)
--- a/modules/getopt-gnu
+++ b/modules/getopt-gnu
@@ -9,6 +9,11 @@ getopt-posix
 
 configure.ac:
 gl_FUNC_GETOPT_GNU
+if test $REPLACE_GETOPT = 1; then
+  AC_LIBOBJ([getopt])
+  AC_LIBOBJ([getopt1])
+  gl_PREREQ_GETOPT
+fi
 gl_MODULE_INDICATOR_FOR_TESTS([getopt-gnu])
 
 Makefile.am:

diff --git a/modules/getopt-posix b/modules/getopt-posix
index c99f8a4feb687b39fc26815234b67263c239e0bc..5da0b472ea752597c6de34b6001235113f1728a9 100644 (file)
--- a/modules/getopt-posix
+++ b/modules/getopt-posix
@@ -13,10 +13,15 @@ unistd
 extensions
 include_next
 arg-nonnull
-gettext-h       [test $GNULIB_UNISTD_H_GETOPT = 1]
+gettext-h       [test $REPLACE_GETOPT = 1]
 
 configure.ac:
 gl_FUNC_GETOPT_POSIX
+if test $REPLACE_GETOPT = 1; then
+  AC_LIBOBJ([getopt])
+  AC_LIBOBJ([getopt1])
+  gl_PREREQ_GETOPT
+fi
 
 Makefile.am:
 BUILT_SOURCES += $(GETOPT_H)