[Webkit-unassigned] [Bug 26734] bugzilla-tool land-and-update needs to fix reviewers in ChangeLogs
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jun 29 13:39:09 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=26734
------- Comment #4 from eric at webkit.org 2009-06-29 13:39 PDT -------
(In reply to comment #3)
> (From update of attachment 31962 [review])
> A few minor things to fix. Free to fix up and submit (unless you do larger
> changes and want another review of it).
Thanks!
> > diff --git a/WebKitTools/Scripts/bugzilla-tool b/WebKitTools/Scripts/bugzilla-tool
> > @@ -86,6 +87,11 @@ def latest_changelog_entry(changelog_path):
> > +def set_reviewer_in_changelog(changelog_path, reviewer):
> > + # inplace=1 creates a backup file and re-directs stdout to the file
> > + for line in fileinput.FileInput(changelog_path, inplace=1):
> > + print line.replace("NOBODY (OOPS!)", reviewer),
>
> Two comments:
> 1. Indent off by one space ("print line....")
> 2. Why is there a comma at the end of this line? (Can it be removed?)
1. Fixed
2. To make print not add a newline when printing:
http://www.python.org/doc/2.4.4/ref/print.html
A "\n" character is written at the end, unless the print statement ends with a
comma. This is the only action if the statement contains just the keyword
print.
> > class LandAndUpdateBug(Command):
> > def execute(self, options, args, tool):
> > + self.update_changelogs_with_reviewer(options.reviewer, bug_id, tool)
> > +
>
> It seems like it should update the date in the changelog as well. Could be a
> fix me for now.
Yeah. Will add a FIXME for now.
> > + else:
> > + log(comment_text)
>
> If the comment contains characters like \n, then I think it will mess up the
> output. Maybe the log function should be updated to look like this
>
> def log(string):
> print >> sys.stderr, "%s" % string
log("foo\nbar") seems to work just fine?
Thanks for the review!
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list