From e4b359d9ae6f76c8d565560e026adc7001b9b9aa Mon Sep 17 00:00:00 2001 From: "Owen W. Taylor" Date: Tue, 18 Nov 2008 22:32:35 -0500 Subject: Don't include the URL in the attachment comment Strip out the URL we add with -u when using the body of the commit as the comment for an attachmnet; it's completely redundant to have an URL pointing back to the bug page itself. --- TODO | 6 ------ git-bz | 10 +++++++++- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/TODO b/TODO index 043cb42..0a529a8 100644 --- a/TODO +++ b/TODO @@ -6,12 +6,6 @@ any intention of working on it myself. - Owen -For attach -u don't include URL in the bugzilla comments - - For 'git bz attach -u', it's just noise to have the URL in the - bugzilla comment; simple: use the body *before* we add the URL - as the comment, not the body after. - Allow editing comment used for attachments When attaching a revised version of a patch, you really want to be diff --git a/git-bz b/git-bz index c1493ac..d5b14e8 100755 --- a/git-bz +++ b/git-bz @@ -735,6 +735,14 @@ def do_apply(bug_reference): commits = git.Commit.find_all(global_repo, "HEAD^!") add_url(bug, commits) +def strip_bug_url(bug, commit_body): + # Strip off the trailing bug URLs we add with -u; we do this before + # using commit body in as a comment; doing it by stripping right before + # posting means that we are robust against someone running add-url first + # and attach second. + pattern = "\s*" + re.escape(bug.get_url()) + "\s*$" + return re.sub(pattern, "", commit_body) + def attach_commits(bug, commits, include_comments=True): # We want to attach the patches in chronological order commits = list(commits) @@ -744,7 +752,7 @@ def attach_commits(bug, commits, include_comments=True): filename = make_filename(commit.message) + ".patch" patch = get_patch(commit) if include_comments: - body = get_body(commit) + body = strip_bug_url(bug, get_body(commit)) else: body = None bug.create_patch(commit.message, body, filename, patch) -- cgit v1.2.3