Changeset 51462 in webkit
- Timestamp:
- Nov 28, 2009 4:17:33 PM (14 years ago)
- Location:
- trunk/WebKitTools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebKitTools/ChangeLog
r51460 r51462 1 2009-11-28 Adam Barth <abarth@webkit.org> 2 3 Reviewed by Eric Seidel. 4 5 [bzt] style-queue should report style errors to bugzilla 6 https://bugs.webkit.org/show_bug.cgi?id=31945 7 8 Currently, we're just logging the style errors locally. With this 9 patch we'll actually log the errors to bugzilla. Note: I plan to run 10 with the "local-only" logging during development. 11 12 * Scripts/modules/commands/queues.py: 13 * Scripts/modules/scm.py: 14 1 15 2009-11-28 Adam Barth <abarth@webkit.org> 2 16 -
trunk/WebKitTools/Scripts/modules/commands/queues.py
r51460 r51462 203 203 self._patches.done(patch) 204 204 205 @classmethod 206 def handle_script_error(cls, tool, patch, script_error): 207 command = script_error.script_args 208 if type(command) is list: 209 command = command[0] 210 # FIXME: We shouldn't need to use a regexp here. ScriptError should 211 # have a better API. 212 if re.search("check-webkit-style", command): 213 message = "Attachment %s did not pass %s:\n\n%s" % (patch["id"], cls.name, script_error.message_with_output(output_limit=None)) 214 # Local-only logging helpful for development: 215 # log("** BEGIN BUG POST **\n%s** END BUG POST **" % message) 216 tool.bugs.post_comment_to_bug(patch["bug_id"], message) 217 205 218 206 219 class BuildQueue(AbstractTryQueue): -
trunk/WebKitTools/Scripts/modules/scm.py
r51437 r51462 96 96 def message_with_output(self, output_limit=500): 97 97 if self.output: 98 if len(self.output) > output_limit:98 if output_limit and len(self.output) > output_limit: 99 99 return "%s\nLast %s characters of output:\n%s" % (self, output_limit, self.output[-output_limit:]) 100 100 return "%s\n%s" % (self, self.output)
Note: See TracChangeset
for help on using the changeset viewer.