Changeset 51259 in webkit
- Timestamp:
- Nov 20, 2009 2:51:56 PM (14 years ago)
- Location:
- trunk/WebKitTools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebKitTools/ChangeLog
r51257 r51259 1 2009-11-20 Adam Barth <abarth@webkit.org> 2 3 Reviewed by Eric Seidel. 4 5 Move prepare_clean_working_directory into the LandingSequence 6 https://bugs.webkit.org/show_bug.cgi?id=31743 7 8 * Scripts/bugzilla-tool: 9 * Scripts/modules/landingsequence.py: 10 1 11 2009-11-20 Yael Aharon <yael.aharon@nokia.com> 2 12 -
trunk/WebKitTools/Scripts/bugzilla-tool
r51253 r51259 191 191 ConditionalLandingSequence.__init__(self, patch, options, tool) 192 192 193 def update(self): 194 pass 195 196 def apply_patch(self): 197 pass 198 199 def close_patch(self, commit_log): 200 self._comment_test = bug_comment_from_commit_text(self._tool.scm(), commit_log) 201 # There is no patch to close. 202 203 def close_bug(self): 193 def run(self): 194 self.build() 195 self.test() 196 commit_log = self.commit() 197 self.close_bug(commit_log) 198 199 def close_bug(self, commit_log): 200 comment_test = bug_comment_from_commit_text(self._tool.scm(), commit_log) 204 201 bug_id = self._patch["bug_id"] 205 202 if bug_id: 206 203 log("Updating bug %s" % bug_id) 207 204 if self._options.close_bug: 208 self._tool.bugs.close_bug_as_fixed(bug_id, self._comment_test)205 self._tool.bugs.close_bug_as_fixed(bug_id, comment_test) 209 206 else: 210 207 # FIXME: We should a smart way to figure out if the patch is attached 211 208 # to the bug, and if so obsolete it. 212 self._tool.bugs.post_comment_to_bug(bug_id, self._comment_test)209 self._tool.bugs.post_comment_to_bug(bug_id, comment_test) 213 210 else: 214 log( self._comment_test)211 log(comment_test) 215 212 log("No bug id provided.") 216 213 … … 305 302 306 303 def run(self): 304 self.clean() 307 305 self.update() 308 306 self.apply_patch() … … 323 321 # Check the tree status first so we can fail early. 324 322 WebKitLandingScripts.ensure_builders_are_green(tool.buildbot, options) 325 WebKitLandingScripts.prepare_clean_working_directory(tool.scm(), options)326 323 327 324 def _process_patch(self, patch, options, args, tool): … … 340 337 # Check the tree status first so we can fail early. 341 338 WebKitLandingScripts.ensure_builders_are_green(tool.buildbot, options) 342 WebKitLandingScripts.prepare_clean_working_directory(tool.scm(), options)343 339 344 340 def _process_patch(self, patch, options, args, tool): 345 341 sequence = ConditionalLandingSequence(patch, options, tool) 346 342 sequence.run_and_handle_errors() 343 347 344 348 345 class LandAttachment(AbstractPatchLandingCommand): -
trunk/WebKitTools/Scripts/modules/landingsequence.py
r51253 r51259 44 44 45 45 def run(self): 46 self.clean() 46 47 self.update() 47 48 self.apply_patch() … … 66 67 self._tool.bugs.reject_patch_from_commit_queue(self._patch["id"], e.message_with_output()) 67 68 WorkQueue.exit_after_handled_error(e) 69 70 def clean(self): 71 WebKitLandingScripts.prepare_clean_working_directory(self._tool.scm(), self._options) 68 72 69 73 def update(self):
Note: See TracChangeset
for help on using the changeset viewer.