Merge lp:~cr3/checkbox/959463 into lp:checkbox

Proposed by Marc Tardif
Status: Merged
Merged at revision: 1332
Proposed branch: lp:~cr3/checkbox/959463
Merge into: lp:checkbox
Diff against target: 36 lines (+4/-4)
2 files modified
debian/changelog (+1/-0)
plugins/apport_prompt.py (+3/-4)
To merge this branch: bzr merge lp:~cr3/checkbox/959463
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+98228@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Looks good, merging. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-03-19 14:53:43 +0000
+++ debian/changelog 2012-03-19 16:34:05 +0000
@@ -31,6 +31,7 @@
3131
32 [Marc Tardif]32 [Marc Tardif]
33 * Linted qt_interface which had a few syntax errors (LP: #949957)33 * Linted qt_interface which had a few syntax errors (LP: #949957)
34 * Fixed apport integration was producing a trace (LP: #959463)
3435
35 [Daniel Manrique]36 [Daniel Manrique]
36 * Bumpted revision number to 0.13.5 in trunk37 * Bumpted revision number to 0.13.5 in trunk
3738
=== modified file 'plugins/apport_prompt.py'
--- plugins/apport_prompt.py 2011-12-15 15:56:02 +0000
+++ plugins/apport_prompt.py 2012-03-19 16:34:05 +0000
@@ -125,7 +125,7 @@
125 self.interface.show_progress_stop()125 self.interface.show_progress_stop()
126126
127 def ui_present_report_details(self, *args):127 def ui_present_report_details(self, *args):
128 return "full"128 return dict(report=1)
129129
130 def ui_question_choice(self, text, options, multiple):130 def ui_question_choice(self, text, options, multiple):
131 self.interface.show_progress_stop()131 self.interface.show_progress_stop()
@@ -224,9 +224,8 @@
224 return224 return
225225
226 if test.get("suite"):226 if test.get("suite"):
227 failed_test_message = _("Test %(name)s from suite %(suite)s failed.") % {227 failed_test_message = _("Test %(name)s failed.") % {
228 'name' : test["name"], 228 'name' : test["name"]}
229 'suite:' : test["suite"]}
230 else:229 else:
231 failed_test_message = _("Test %s failed.") % test["name"]230 failed_test_message = _("Test %s failed.") % test["name"]
232 failed_test_message += "\n" + _("Do you want to report a bug?")231 failed_test_message += "\n" + _("Do you want to report a bug?")

Subscribers

People subscribed via source and target branches