diff options
author | Will Thompson <will.thompson@collabora.co.uk> | 2011-07-27 16:22:36 +0100 |
---|---|---|
committer | Will Thompson <will.thompson@collabora.co.uk> | 2011-07-27 16:22:36 +0100 |
commit | f50a1e625217d528c6a6406d6ce610773910d1d8 (patch) | |
tree | 52616c67cd4f62de4d8a6bee5afcb0408c29f10b | |
parent | 7f64afa396c7e639e7920a60d46b980a12d5c2ee (diff) |
make-release-mail: detect headers more robustly.telepathy-gabble-0.13.3
I just broke this by having a line beginning 'telepathy-gabble' which
was not actually a version header.
-rw-r--r-- | tools/make-release-mail.py | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/tools/make-release-mail.py b/tools/make-release-mail.py index 5c42b4798..b03ebd256 100644 --- a/tools/make-release-mail.py +++ b/tools/make-release-mail.py @@ -6,6 +6,15 @@ # to <telepathy@lists.freedesktop.org>. I hope that you enjoy your stay. import sys +import re + +def looks_like_a_header(line, package, version=None): + if version is None: + pattern = "^%s .* \(.*\)$" % package + else: + pattern = "^%s %s \(.*\)$" % (package, version) + + return re.match(pattern, line) is not None def extract_description(package, version, news_path): release_name = [] @@ -15,7 +24,7 @@ def extract_description(package, version, news_path): lines = (line for line in f.readlines()) for line in lines: # Find the 'telepathy-foo 0.1.2' header - if line.startswith("%s %s" % (package, version)): + if looks_like_a_header(line, package, version): break # Skip the ====== line, and the first blank line @@ -27,7 +36,7 @@ def extract_description(package, version, news_path): for line in lines: line = line.rstrip() # If we hit the next version header, we're done - if line.startswith(package): + if looks_like_a_header(line, package): break # Else, if we hit a blank line and we're still reading the release # name, we're done with the release name. |