Closed Bug 975312 Opened 6 years ago Closed 6 years ago
.check _for _crashes() should support save _path and quiet parameters
Right now the check_for_crashes() method of the Runner class does not support the save_path parameter of mozcrash. We have to get this added so that Mozmill can use it on bug 619204.
Test frameworks might not always want to let the information printed to stdout, but handle it on their own. So also forwarding the quiet parameter sounds important to me.
Summary: [mozrunner] Runner.check_for_crashes() should support save_path for storing minidump files → [mozrunner] Runner.check_for_crashes() should support save_path and quiet parameters
I also pushed this to try: https://tbpl.mozilla.org/?tree=Try&rev=00aa990af6bf
Included the mozrunner version bump too.
Comment on attachment 8379660 [details] [diff] [review] Patch v1.1 Review of attachment 8379660 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thanks!
Attachment #8379660 - Flags: review?(ahalberstadt) → review+
Target Milestone: --- → mozilla30
6 years ago
Damn. Somehow I pushed the wrong patch to inbound which didn't include the version bump. :/ I will most likely do it later or wait until I'm back.
Pushed version bump to 5.35: https://hg.mozilla.org/integration/mozilla-inbound/rev/f42032325f18
(In reply to Henrik Skupin (:whimboo) [away 02/24 - 02/28] from comment #8) > Pushed version bump to 5.35: > https://hg.mozilla.org/integration/mozilla-inbound/rev/f42032325f18 Not sure what went wrong here, but it ended-up as a no-op. :( So I pushed it again: https://hg.mozilla.org/integration/mozilla-inbound/rev/c91a96c71e62 Given that we haven't seen any problems with the former patch, I'm going to release mozrunner 5.35 now.
Submitting dist/mozrunner-5.35.tar.gz to http://pypi.python.org/pypi Server response (200): OK
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.