Hi Dmitriy,

> I think I've recently found a small bug inside old gnulib-tool.
> As you know we parse m4/gnulib-cache.m4 file to get cached variables
> which we want to use in import modes. There is gl_WITNESS_C_MACRO field,
> which we parse to get witness_c_macro variable. But at line 5143, where
> we generate a new gnulib-cache.m4 file, I see the following:
> 
> echo "gl_WITNESS_C_DOMAIN([$witness_c_macro])"
> 
> Have I misunderstood something or there is a small bug? May be the
> cause of the bug is just copy-pasting of the previous line, where we do
> the same thing with gl_PO_DOMAIN.

Of course, yes, you're 100% right. Fixed:


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

        gnulib-tool: Fix persistence of --witness-c-macro option.
        * gnulib-tool (func_import): Fix typo in emit of gl_WITNESS_C_MACRO.
        Reported by Dmitriy Selyutin <[email protected]>.

--- gnulib-tool.orig    Mon Aug 13 12:48:12 2012
+++ gnulib-tool Mon Aug 13 12:46:45 2012
@@ -5140,7 +5140,7 @@
     fi
     echo "gl_MACRO_PREFIX([$macro_prefix])"
     echo "gl_PO_DOMAIN([$po_domain])"
-    echo "gl_WITNESS_C_DOMAIN([$witness_c_macro])"
+    echo "gl_WITNESS_C_MACRO([$witness_c_macro])"
     if test -n "$vc_files"; then
       echo "gl_VC_FILES([$vc_files])"
     fi


Reply via email to