maint: patch gitlog-to-changelog to reduce bogus warnings.

Rather than trying to find a new useful warning among the 55
expected warnings from vanilla gitlog-to-changelog, patch it to
elide the bogus warnings from duplicate log entries.
* gl/bulid-aux/gitlog-to-changelog.diff: Omit-from-ChangeLog is
a new log entry tag to skip legitimate log duplicates.
* build-aux/git-log-fix: Instead of deleting all content of
duplicates and generating a warning, append the new
Omit-from-ChangeLog tag to elide silently.
* Makefile.am *($(changelog)): Remove the 'expect 55 warnings'
echo.

Signed-off-by: Gary V. Vaughan <gary@gnu.org>
This commit is contained in:
Gary V. Vaughan 2013-01-03 16:49:35 +07:00
parent a5a4944fbb
commit 2d96bc612c
3 changed files with 87 additions and 61 deletions

View File

@ -536,7 +536,6 @@ $(changelog): FORCE
rm -f '$@'; mv '$@T' '$@'; \
cat '$(changelog_old)' >> '$@'; \
fi
@echo 'NOTE: 55 empty commit log warnings above are expected'
## Arrange so that .tarball-version appears only in the distribution
## tarball, and never in a checked-out repository.

View File

@ -62,163 +62,163 @@ s|\n\n|\n|g
## branch.
# Date: Tue Oct 2 21:05:53 2012 +0700
4440d6523bf9c668720f9ffa66fb89e91912a504
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
0fc275469ef020177d8dff2c041456f6b6155537
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
692bc68dc9a580647a164d6338ca7e13cb593966
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
6cb0d0d771bc1e3be402929b900000ca7a8a16d4
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d50e123595d80b5582d0663ad50e3454051c482d
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
e0f66dc54d2f7906a2513c68d76362be4bcbbef3
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
ae68608314993e81836257ac1eec449256decd53
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d576fa9da144f2e9ab14337c9590d4abfdea985f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
3cbc0e26f0a7f0bbea559912c3da920b4ead8e5e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
83d52fd6d2039c048f731133bec0ae8a1bf9382d
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
1e70622d7c127b4dd90b3a5b0650d323ec2305fc
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
1fcb9bc4e4b7b7166a6f45da633f813faee6e00f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
bf6aa3082089d12bf4500b469c096263823a75b4
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
43acd6749a29f079452cfb70b6c6e1c8f1469128
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
8c6e33197e36463cb7fcf5d575a53c3f414ec4ff
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
7db681a2696c75dc6c55261ea8fd11f92e42db2b
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
59f8a3a6e1a26d501e5865f7611d8828aac5579c
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
f2a640f7e03e5dcf1510833a0e1e4991032b5f96
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
90529a55ef40653051730d3c20203fa2ad5e8f7f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d702946c65b309c389ae717a4d2d6fe158c3ee5f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
8392cd263e847a22b2409c35bf03d7c9e68d2cd0
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
430e782755bbe8061a5e7f9d856b33e3b7da3ac8
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d60b4387c178100b0bd3835c61d0f4771330bf6a
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
47a889a4ca2009e2147fa5b2f95baefe1dca8288
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
8f975a1368594126e37d85511f1f96164e466d93
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
f8a4b90f9a712f47b7627d2cf96eed74825df91f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
b25625e6b25427dba80051b2924060b1e84ea73e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
3467227ccb6c99f4c2122b9fb3fa67d54b5bd54b
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
8bfac9e3e82646868907aede4ebe4deb158a2e8a
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
baffe661910947df0dcd5a5bbd6512fc0bce59b9
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
822881fe3c69ca474b8e13d60243dd9c1a1c3fea
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
f7f249a0e6b5093b8752f1ba863345bfcd5a237e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
c03789b8ee33f136d3bbd7f7a111ad863bb4c7e8
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
78385ffd8af73b927549c3596da67233f2f92f0e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
7d25536e80b68042a6d999c11a0c5c9dc948f82e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
7a2a8f32bb83d6b16db8436b530ec5e88fbd0751
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
c5dec32e37e90ce46813efe7dfe8c5500cc55bbc
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
a2445f0fdbeaa72c18a893714e4437ea01faa08a
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
0c953ba5cc71bd1de615df28fbf16e27be62244e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
f32a760fbd40796be0b7e3be41226a82db1748d4
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
a80878742dfd0f5127c1ddc5f803fab980eb816a
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d56683630a6415e59f1be7a6540c4733e9671bd8
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
ca821436009a93c2e8f3c5ce07ed23d90bbba49e
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
c7d7e46fbea7d6249680da71ecda1d84bdec858a
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
156a851bc5d758f2d4d7fd89cef678ae72f35e77
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
87c4cffb19cf5ba8dc088e0ad357e5c3d4be2433
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
c5230035e5df31976285bf36141d89236416230d
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
cb4ed2559e89354ea846a686543206d3445ea5ec
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
9f3cec5b2ec476bd4fd978c1e23398b25d36cd2f
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
5f27eeb0a37f15b32ccb1b97b2ed9022f5df9a60
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
013c91874e974543d2f58843c1e839ba102d6ee0
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
006695fed2e9cd10664c97c69880a82545dde1ef
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
ff3b32aed7db925a7390d92ffede055b4c8ccf18
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
d43b40113ec1e8289e48d77f00e8947f2fd98fd4
# Date: Tue Oct 2 21:02:16 2012 +0700
@ -284,7 +284,7 @@ s|\n(Improve )|\nlibtool: $1|
9333e74fc7b76a11ed04a19343eb5dd75a1035f3
# Date: Sun Feb 19 16:21:00 2012 -0600
# Elide duplicate log entries.
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
c0c49f289f22ae670066657c60905986da3b555f
# Date: Sun Feb 19 15:33:48 2012 -0600
@ -324,7 +324,7 @@ s|\n(Support )|\nlibtool: $1|
962aa919f51cdf8e2cee4fb2d1d9bafa34d50887
# Date: Fri Nov 25 14:53:42 2011 +0700
# Elide duplicate log entries.
s|\n[\s\S]*$||
s|$|Omit-from-ChangeLog: Yes|
11d16ab06c36948c053c507f5f9e06ff3de221c7
# Date: Fri Nov 25 14:39:57 2011 +0700

View File

@ -1,6 +1,32 @@
--- gnulib/build-aux/gitlog-to-changelog 2012-10-22 15:47:21.000000000 +0700
+++ build-aux/gitlog-to-changelog 2012-10-22 15:47:14.000000000 +0700
@@ -351,26 +351,6 @@
diff --git a/build-aux/gitlog-to-changelog b/build-aux/gitlog-to-changelog
index e02d34c..aeae458 100755
--- a/build-aux/gitlog-to-changelog
+++ b/build-aux/gitlog-to-changelog
@@ -99,6 +99,8 @@ at the beginning of a log message line. They are not copied to the output.
Co-authored-by: Joe User <user\@example.com>
List the specified name and email address on a second
ChangeLog header, denoting a co-author.
+ Omit-from-ChangeLog: Yes
+ No ChangeLog entry is made for this commit.
Signed-off-by: Joe User <user\@example.com>
These lines are simply elided.
@@ -323,10 +325,14 @@ sub git_dir_option($)
strftime ("%F", localtime ($1)), $2;
my @coauthors = grep /^Co-authored-by:.*$/, @line;
+
+ my $omit = grep /^Omit-from-ChangeLog:\s+[Yy]es$/, @line;
+
# Omit meta-data lines we've already interpreted.
@line = grep !/^(?:Signed-off-by:[ ].*>$
|Co-authored-by:[ ]
|Copyright-paperwork-exempt:[ ]
+ |Omit-from-ChangeLog:[ ]
)/x, @line;
# Remove leading and trailing blank lines.
@@ -351,33 +357,33 @@ sub git_dir_option($)
. substr ($_, 5) . "\n";
}
@ -27,9 +53,10 @@
# If there were any lines
if (@line == 0)
{
@@ -378,6 +358,26 @@
warn "$ME: warning: empty commit message:\n $date_line\n";
}
else
- else
+ elsif ( ! $omit )
{
+ # If clustering of commit messages has been disabled, if this header
+ # would be different from the previous date/name/email/coauthors header,