[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: --do-changelog
From: |
Bruno Haible |
Subject: |
Re: --do-changelog |
Date: |
Sun, 26 Aug 2012 17:05:08 +0200 |
User-agent: |
KMail/4.7.4 (Linux/3.1.10-1.9-desktop; KDE/4.7.4; x86_64; ; ) |
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
- --do-changelog, address@hidden, 2012/08/20
- Re: --do-changelog,
Bruno Haible <=