Merge lp:~cmiller/xdiagnose/fix-confusing-dialog into lp:xdiagnose

Proposed by Chad Miller
Status: Merged
Merge reported by: Bryce Harrington
Merged at revision: not available
Proposed branch: lp:~cmiller/xdiagnose/fix-confusing-dialog
Merge into: lp:xdiagnose
Diff against target: 27 lines (+8/-8)
1 file modified
apport/source_xorg.py (+8/-8)
To merge this branch: bzr merge lp:~cmiller/xdiagnose/fix-confusing-dialog
Reviewer Review Type Date Requested Status
Bryce Harrington code Approve
Review via email: mp+82545@code.launchpad.net

Commit message

Clarify the 'development is completed' dialog.

To post a comment you must log in.
Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks, looks good.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'apport/source_xorg.py'
2--- apport/source_xorg.py 2011-10-05 18:50:03 +0000
3+++ apport/source_xorg.py 2011-11-17 15:03:26 +0000
4@@ -211,15 +211,15 @@
5 elif status == "Supported" or status == "Current Stable Release":
6 # TODO: Perhaps avoid this check if the time since the release < 1 month?
7 response = ui.choice(
8- "Development is completed for the '%s' version of Ubuntu, so\n"
9- "you should probably use technical support channels unless you know for\n"
10- "certain it should be reported here?" %(distro_codename),
11+ "Since development is completed for the '%s' version of Ubuntu, \n"
12+ "you should probably use its technical support channels instead \n"
13+ "of this tool.\n" %(distro_codename),
14 [
15- "I don't know",
16- "Yes, I already know the fix for this problem.",
17- "Yes, The problem began right after doing a system software update.",
18- "Yes.",
19- "No, please point me to a good place to get support.",
20+ "I don't know what to do",
21+ "Continue. I already know the fix for this problem.",
22+ "Continue. The problem began right after system software update.",
23+ "Continue. I was referred here by Support.",
24+ "Please point me to a good place to get support.",
25 ]
26 )
27 if response == None:

Subscribers

People subscribed via source and target branches

to all changes: