[SRU][P, T, X, B, C][PATCH v2] UBUNTU: [Packaging] git-ubuntu-log -- fix order of tracking bug commit

Kleber Souza kleber.souza at canonical.com
Tue Oct 2 10:33:13 UTC 2018


On 09/21/18 13:37, Kleber Sacilotto de Souza wrote:
> The package tracking bug number is not inserted anymore on the changelog
> by the link-to-tracker script. Instead, it creates/updates the
> debian.<branch>/tracking-bug file and commits the change.
> 
> We want this commit to be added to the changelog in a similar fashion as
> before, as the first entry. So change the git-ubuntu-log script to
> detect that commit and insert it in the right place.
> 
> Ignore: yes
> Signed-off-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>
> ---
>  debian/scripts/misc/git-ubuntu-log | 27 +++++++++++++++++++--------
>  1 file changed, 19 insertions(+), 8 deletions(-)
> 
> diff --git a/debian/scripts/misc/git-ubuntu-log b/debian/scripts/misc/git-ubuntu-log
> index 490ecce73c48..4bf738f8fed1 100755
> --- a/debian/scripts/misc/git-ubuntu-log
> +++ b/debian/scripts/misc/git-ubuntu-log
> @@ -28,13 +28,19 @@ def add_entry(entry):
>              else:
>                  combo = '__mainline__'
>          else:
> -            if combo not in keys:
> -                keys.append(combo)
> +            if entry.get('subject', "") == 'UBUNTU: link-to-tracker: update tracking bug':
> +                # Construct a key with '__trackingbug__' on the first position
> +                # and the tracking bug number afterwards
> +                combo.insert(0, '__trackingbug__')
> +                # Tracking bug goes at the top
> +                keys.insert(0, combo)
> +            else:
> +                if combo not in keys:
> +                    keys.append(combo)
>  
>          entry['key'] = combo
>          entries.append(entry)
>  
> -
>  # Suck up the git log output and extract the information we need.
>  keys = []
>  entry = None
> @@ -90,6 +96,10 @@ for key in keys:
>          for bug in key:
>              if bug.startswith('CVE-'):
>                  title_set.append(bug)
> +            elif bug == '__trackingbug__':
> +                # Look for the tracking bug number on the second
> +                # position of the key
> +                continue
>              else:
>                  bug_info = None
>  
> @@ -132,8 +142,9 @@ for key in keys:
>  
>              emit_title = False
>  
> -        title_lines = textwrap.wrap(entry['subject'], 76)
> -        print('    - ' + title_lines[0])
> -        for line in title_lines[1:]:
> -            line = line.replace('LP###', 'LP: #')
> -            print('      ' + line)
> +        if key[0] != '__trackingbug__':
> +            title_lines = textwrap.wrap(entry['subject'], 76)
> +            print('    - ' + title_lines[0])
> +            for line in title_lines[1:]:
> +                line = line.replace('LP###', 'LP: #')
> +                print('      ' + line)
> 

Please note that this patch still needs to be applied to Cosmic.

Thanks,
Kleber




More information about the kernel-team mailing list