Hello Dmitriy,

> it seems I've found another very small bug inside gnulib-tool. See line 3920 
> of emit_autoconf_snippet: it will never do the right indentation (which we 
> need e.g. in emit_autoconf_snippets) since $indentation is always empty. We 
> just need to add one line in the beginning of emit_autoconf_snippet:
> 
> indentation=$1

You're entirely right again. Fixed like you suggested:


2012-08-14  Bruno Haible  <[email protected]>

        gnulib-tool: Fix indentation of generated gnulib-comp.m4 file.
        * gnulib-tool (func_emit_autoconf_snippet): Initialize indentation
        from argument.
        Reported and fix suggested by Dmitriy Selyutin <[email protected]>.

--- gnulib-tool.orig    Wed Aug 15 01:28:53 2012
+++ gnulib-tool Wed Aug 15 01:26:50 2012
@@ -3892,6 +3892,7 @@
 # func_emit_autoconf_snippet indentation
 # emits the autoconf snippet of a module.
 # Input:
+# - indentation       spaces to prepend on each line
 # - local_gnulib_dir  from --local-dir
 # - modcache          true or false, from --cache-modules/--no-cache-modules
 # - sed_replace_build_aux  sed expression that replaces reference to build-aux
@@ -3905,9 +3906,9 @@
 #                     command line options.
 # - disable_gettext   true or false. It tells whether to disable AM_GNU_GETTEXT
 #                     invocations.
-# - indentation       spaces to prepend on each line
 func_emit_autoconf_snippet ()
 {
+  indentation="$1"
   if { case $module in
          gnumakefile | maintainer-makefile)
            # These modules are meant to be used only in the top-level 
directory.


Reply via email to