gitlog-to-changelog: treat a message with only blank lines as empty

* build-aux/gitlog-to-changelog: Move the code that removes leading and
trailing blank lines before the code that issues a warning about an
empty commit message.

diff --git a/ChangeLog b/ChangeLog
index 603a16f..30cc2af 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-10-29  Dmitry V. Levin  <ldv@altlinux.org>
+
+       gitlog-to-changelog: treat a message with only blank lines as empty.
+       * build-aux/gitlog-to-changelog: Move the code that removes leading and
+       trailing blank lines before the code that issues a warning about an
+       empty commit message.
+
 2011-10-30  Jim Meyering  <meyering@redhat.com>
 
        test-parse-datetime.c: avoid new DST-related false positive test failure

diff --git a/build-aux/gitlog-to-changelog b/build-aux/gitlog-to-changelog
index 4559704..a5fd80d 100755 (executable)
--- a/build-aux/gitlog-to-changelog
+++ b/build-aux/gitlog-to-changelog
@@ -152,6 +152,10 @@ sub quoted_cmd(@)
       # Omit "Signed-off-by..." lines.
       @line = grep !/^Signed-off-by: .*>$/, @line;
 
+      # Remove leading and trailing blank lines.
+      while ($line[0] =~ /^\s*$/) { shift @line; }
+      while ($line[$#line] =~ /^\s*$/) { pop @line; }
+
       # If there were any lines
       if (@line == 0)
         {
@@ -159,10 +163,6 @@ sub quoted_cmd(@)
         }
       else
         {
-          # Remove leading and trailing blank lines.
-          while ($line[0] =~ /^\s*$/) { shift @line; }
-          while ($line[$#line] =~ /^\s*$/) { pop @line; }
-
           # Prefix each non-empty line with a TAB.
           @line = map { length $_ ? "\t$_" : '' } @line;