[kteam-tools][PATCH] verify-release-ready: handle unparseable master release tracking bug

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[kteam-tools][PATCH] verify-release-ready: handle unparseable master release tracking bug

Kamal Mostafa-2
For derivatives, better handle the situation when no release tracking bug
can be parsed from the master changelog entry (e.g. a manually constructed
entry missing the usual "* Release Tracking Bug" block).

In this case, emit
  "master release tracking bug found: warning"
instead of incorrectly complaining that tracking bugs are not unique.

Signed-off-by: Kamal Mostafa <[hidden email]>
---
 maintscripts/verify-release-ready | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/maintscripts/verify-release-ready b/maintscripts/verify-release-ready
index 08d9121..5eb2cfb 100755
--- a/maintscripts/verify-release-ready
+++ b/maintscripts/verify-release-ready
@@ -249,6 +249,7 @@ class VerifyReleaseReady():
             if branch not in ['master', 'master-next'] and not branch.endswith('-next'):
                 master_changelog = Debian.master_changelog()
                 found_tracker = False
+                master_tracker_id='NOT-FOUND'
                 l = 0
                 for line in master_changelog[1]['content']:
                     l += 1
@@ -259,13 +260,11 @@ class VerifyReleaseReady():
                     if 'Tracking Bug' in line:
                         found_tracker = True
 
-                unique_tracker = False
                 if found_tracker:
-                    if tracker_id != master_tracker_id:
-                        unique_tracker = True
-
-
-                s.status('unique release tracking bug', unique_tracker)
+                    unique_tracker = tracker_id != master_tracker_id
+                    s.status('unique release tracking bug', unique_tracker)
+                else:
+                    s.status('master release tracking bug found', 'warning')
 
         # Verify that the changelog entry contains no "Miscellaneous" entries
         s.verify_changelog_nomisc()
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [kteam-tools][PATCH] verify-release-ready: handle unparseable master release tracking bug

brad.figg
Reply | Threaded
Open this post in threaded view
|

ACK: [kteam-tools][PATCH] verify-release-ready: handle unparseable master release tracking bug

Marcelo Henrique Cerri
In reply to this post by Kamal Mostafa-2
--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (484 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Applied: [kteam-tools][PATCH] verify-release-ready: handle unparseable master release tracking bug

Kamal Mostafa-2
In reply to this post by Kamal Mostafa-2