CMT/ACK: [SRU X/B/F/G/H/U] UBUNTU: [Packaging] insertchanges: avoid double newline

[Kelsey Skunberg]

Before applying could we get a BugLink to include?

Otherwise lgtm.

Acked-by: Kelsey Skunberg <kelsey.skunberg at canonical.com>

On 2020-11-05 13:16:43 , Thadeu Lima de Souza Cascardo wrote:
> When some changes have been already added to the changelog, like when using
> insert-ubuntu-changes, and there are no other changes, we end up with two
> newlines right after the stanza header.
> 
> Add a $skip_newline variable that allows us to skip that extra newline when
> there are no other changes.
> 
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo at canonical.com>
> ---
>  debian/scripts/misc/insert-changes.pl | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/debian/scripts/misc/insert-changes.pl b/debian/scripts/misc/insert-changes.pl
> index c820597a9fc9..dde210faac34 100755
> --- a/debian/scripts/misc/insert-changes.pl
> +++ b/debian/scripts/misc/insert-changes.pl
> @@ -13,17 +13,26 @@ open(CHANGES, "< $debian/changes") or die "Cannot open new changes";
>  open(NEW, "> $debian/changelog.new") or die "Cannot open new changelog";
>  
>  $printed = 0;
> +my $skip_newline = 0;
>  
>  while (<CHANGELOG>) {
>  	if (/^  CHANGELOG: /) {
>  		next if $printed;
>  
> +		$skip_newline = 1;
>  		while (<CHANGES>) {
> +			$skip_newline = 0;
>  			print NEW;
>  		}
>  
>  		$printed = 1;
>  	} else {
> +		if (/^$/) {
> +			if ($skip_newline == 1) {
> +				$skip_newline = 0;
> +				next;
> +			}
> +		}
>  		print NEW;
>  	}
>  }
> -- 
> 2.27.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team