Re: --do-changelog

[Bruno Haible]

Hi Dmitriy,

> it seems that I've recently found a new small bug inside gnulib-tool.
> We have --do-changelog, but it is never used. It is only set when we
> process command-line arguments, but I couldn't find any piece of code
> where we test its value. I guess that it lives in gnulib-tool since
> the ancient times when we need such option. Nowadays as I can see it
> is useless.

Yes, in the beginning [1] it was planned that gnulib-tool --add-import
would write ChangeLog entries, like gettextize does. But this practice
has not seen many friends, since most people manage their ChangeLog themselves
(or have it autogenerated).

No one has asked for this feature in several years, therefore it's not
likely that it will be implemented.

Removing the old cruft.

[1] http://lists.gnu.org/archive/html/bug-gnulib/2005-11/msg00001.html


2012-08-26  Bruno Haible  <address@hidden>

        gnulib-tool: Remove no-op option --no-changelog.
        * gnulib-tool (func_usage): Don't mention --no-changelog.
        (do_changelog): Remove variable.
        Reported by Dmitriy Selyutin <address@hidden>.

--- gnulib-tool.orig    Sun Aug 26 16:58:08 2012
+++ gnulib-tool Sun Aug 26 16:57:44 2012
@@ -280,7 +280,6 @@
       --vc-files            Update version control related files.
       --no-vc-files         Don't update version control related files
                             (.gitignore and/or .cvsignore).
-      --no-changelog        Don't update or create ChangeLog files.
 
 Options for --create-[mega]testdir, --[mega]test:
 
@@ -957,7 +956,6 @@
 # - vc_files        true if --vc-files was given, false if --no-vc-files was
 #                   given, blank otherwise
 # - autoconf_minversion  minimum supported autoconf version
-# - do_changelog    false if --no-changelog was given, : otherwise
 # - doit            : if actions shall be executed, false if only to be printed
 # - symbolic        true if --symlink or --more-symlinks was given, blank
 #                   otherwise
@@ -998,7 +996,6 @@
   po_domain=
   witness_c_macro=
   vc_files=
-  do_changelog=:
   doit=:
   symbolic=
   lsymbolic=
@@ -1268,7 +1265,7 @@
         vc_files=false
         shift ;;
       --no-changelog | --no-changelo | --no-changel | --no-change | --no-chang 
| --no-chan | --no-cha | --no-ch )
-        do_changelog=false
+        # A no-op for backward compatibility.
         shift ;;
       --dry-run )
         doit=false
@@ -1337,7 +1334,6 @@
       echo "you need to use 'gnulib --import' - at your own risk!" 1>&2
       func_exit 1
     fi
-    do_changelog=false
   fi
   if test -n "$pobase" && test -z "$po_domain"; then
     echo "gnulib-tool: together with --po-base, you need to specify 
--po-domain" 1>&2