From 33f8dd5f35cb6de5737b9899ec5abe2d7b2cc0ec Mon Sep 17 00:00:00 2001 From: Mark Andrews Date: Wed, 23 May 2018 14:31:15 +1000 Subject: [PATCH] ensure there is a blank line before a changes entry and a release marker (cherry picked from commit c91770549f86f36f62dbbaef43e7a529ff9ae8aa) (cherry picked from commit d87eb5804dee423ccb3984462a6df4b78feb9ec1) --- util/check-changes | 19 +++++++++++++++++-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/util/check-changes b/util/check-changes index 4342c22161..ee79da23b5 100644 --- a/util/check-changes +++ b/util/check-changes @@ -24,11 +24,11 @@ $\ = "\n"; # set output record separator $change = 0; $status = 0; +$wasblank = 1; line: while (<>) { ($Fld1,$Fld2,$Fld3,$Fld4) = split(' ', $_, 9999); - $change = 0 if (!$master && $Fld1 eq "---" && - $Fld3 eq "released" && $Fld4 eq "---"); + $change = 0 if (!$master && /^\s+--- .* ---$/); if ($Fld1 =~ /^[1-9][0-9]*\.$/ && $Fld2 =~ /^\[.*\]$/) { if ($change != 0 && $Fld1 >= $change) { print 'bad change number', $Fld1; @@ -39,6 +39,21 @@ line: while (<>) { $status = 1; } $change = $Fld1; + if (!$wasblank) { + print 'missing blank line before change', $Fld1; + $status = 1; + } + } + if (/^\s+--- .* ---$/) { + if (!$wasblank) { + print 'missing blank line before release marker for', $Fld2; + $status = 1; + } + } + if ($Fld1 eq "") { + $wasblank = 1; + } else { + $wasblank = 0; } } -- GitLab