[kteam-tools][PATCH 0/2] debian.py: fix changelog[content, bugs] off-by-one

Kamal Mostafa kamal at canonical.com
Fri Jun 15 20:44:30 UTC 2018


PATCH 1/2: debian.py: fix changelog[content,bugs] off-by-one

The debian changelog parser in debian.py (both implementations) is bugged
such that the ['content'] and ['bugs'] values for each changelog[N] actually
get placed in changelog[N+1].  I.e. changelog[0] represents the topmost
changelog entry but its changelog[0]['content'] and changelog[0]['bugs']
are always empty; their values end up in changelog[1].

Fix the buggy debian.py changelog parser behavior and verify-release-ready's
expectation of it.

IMPORTANT: Any other out-of-tree users of this debian.py changelog
parser will see a behavior change w.r.t. the ['content'] and ['bugs']
elements.


PATCH 2/2: verify-release-ready: rename internal cl variables

No functional change; rename internal variables for sanity.


 -Kamal

----

Kamal Mostafa (2):
  debian.py: fix changelog[content,bugs] off-by-one
  verify-release-ready: rename internal cl variables

 ktl/debian.py                     |  41 +++++++------
 maintscripts/verify-release-ready | 118 +++++++++++++++++++-------------------
 py3/lib/debian.py                 |  33 ++++++-----
 3 files changed, 102 insertions(+), 90 deletions(-)

-- 
2.7.4





More information about the kernel-team mailing list