[Merge] lp:~sil2100/ubuntu-archive-tools/common-sru-process-bug into lp:ubuntu-archive-tools

Łukasz Zemczak lukasz.zemczak at canonical.com
Mon Nov 20 16:38:25 UTC 2017


Łukasz Zemczak has proposed merging lp:~sil2100/ubuntu-archive-tools/common-sru-process-bug into lp:ubuntu-archive-tools.

Commit message:
Extract the process_bug() function to a common module so that both sru-review and sru-accept use the same mechanics for updating SRU bugs after acceptance. This also slightly changes the behavior of sru-accept for more than one series.

Requested reviews:
  Brian Murray (brian-murray)
  Ubuntu Package Archive Administrators (ubuntu-archive)

For more details, see:
https://code.launchpad.net/~sil2100/ubuntu-archive-tools/common-sru-process-bug/+merge/333980

Extract the process_bug() function to a common module so that both sru-review and sru-accept use the same mechanics for updating SRU bugs after acceptance. This also slightly changes the behavior of sru-accept for more than one series.

Currently we had two versions of process_bug and the sru-accept one was outdated. Having to maintain two of those was crazy.

Also, with this change, sru-accept when called with multiple targets now just runs process_bug() multiple times (e.g. adding one comment per series) instead of once with multiple series in the bug comment. I changed this because it was easier to do and mimics what happens when you use the sru-review tool.
-- 
Your team Ubuntu Package Archive Administrators is requested to review the proposed merge of lp:~sil2100/ubuntu-archive-tools/common-sru-process-bug into lp:ubuntu-archive-tools.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 18169 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-archive/attachments/20171120/36fc9f42/attachment-0001.diff>


More information about the ubuntu-archive mailing list