Merge lp:~sil/ubuntu/lucid/rhythmbox/add-music-store-apport-hook into lp:ubuntu/lucid/rhythmbox

Proposed by Stuart Langridge
Status: Merged
Merge reported by: James Westby
Merged at revision: not available
Proposed branch: lp:~sil/ubuntu/lucid/rhythmbox/add-music-store-apport-hook
Merge into: lp:ubuntu/lucid/rhythmbox
Diff against target: 83 lines (+62/-0) (has conflicts)
2 files modified
debian/changelog (+12/-0)
debian/source_rhythmbox.py (+50/-0)
Text conflict in debian/changelog
Conflict adding file debian/source_rhythmbox.py.  Moved existing file to debian/source_rhythmbox.py.moved.
To merge this branch: bzr merge lp:~sil/ubuntu/lucid/rhythmbox/add-music-store-apport-hook
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+22117@code.launchpad.net

Description of the change

Add to the Rhythmbox apport hook so that:
1. look for webkit or ubuntuone in the stacktrace and if found, assume that this is a music store bug and reassign to rhythmbox-ubuntuone-music-store
2. when asking the user what the problem was, add an option for the problem being with the Music Store and reassign to rhythmbox-ubuntuone-music-store if that was picked.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-03-24 23:38:19 +0000
3+++ debian/changelog 2010-03-25 11:50:37 +0000
4@@ -1,3 +1,4 @@
5+<<<<<<< TREE
6 rhythmbox (0.12.7-0ubuntu9) lucid; urgency=low
7
8 * debian/patches/21_git_ignore_mounts_without_device.patch:
9@@ -196,6 +197,17 @@
10
11 -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 25 Feb 2010 19:14:46 +0100
12
13+=======
14+rhythmbox (0.12.6git20100223-0ubuntu2) lucid; urgency=low
15+
16+ * debian/source_rhythmbox.py:
17+ - apport hook for rhythmbox (lp: #525888)
18+ * debian/rhythmbox.install:
19+ - update to install apport hook
20+
21+ -- Nigel Babu <nigelbabu@gmail.com> Thu, 25 Feb 2010 09:49:44 +0530
22+
23+>>>>>>> MERGE-SOURCE
24 rhythmbox (0.12.6git20100223-0ubuntu1) lucid; urgency=low
25
26 * New git snapshot:
27
28=== added file 'debian/source_rhythmbox.py'
29--- debian/source_rhythmbox.py 1970-01-01 00:00:00 +0000
30+++ debian/source_rhythmbox.py 2010-03-25 11:50:37 +0000
31@@ -0,0 +1,50 @@
32+import os
33+import re
34+
35+import apport.packaging
36+import apport.hookutils
37+
38+def mask_string (str):
39+ MASK = '##MASKED##'
40+ return str.group(1) + MASK
41+
42+def mask_values(gconfinfo):
43+ """ strip personal/private information from the GConf entries """
44+ pattrn = re.compile ('((add_dir|library_locations|download_prefix|share_password|share_name|username|password)=)(.*)$',
45+ re.IGNORECASE)
46+ newReport = ""
47+ for line in gconfinfo.splitlines():
48+ line = pattrn.sub (mask_string, line)
49+ newReport += line + '\n'
50+ return newReport
51+
52+def add_info(report, ui):
53+
54+ # Check to see if the stacktrace is a webkit or libubuntuone issue
55+ # if it is, then reassign to the Ubuntu One Music Store
56+ stacktrace = report.get("Stacktrace", None)
57+ if stacktrace:
58+ if "ubuntuone" in stacktrace or "webkit" in stacktrace:
59+ report.add_package_info("rhythmbox-ubuntuone-music-store")
60+ return
61+
62+
63+ response = ui.choice("How would you describe the issue?", ["The rhythmbox interface is not working correctly", "No sound is being played", "Some audio files are not being played correctly", "The Ubuntu One Music Store is not working correctly"], False)
64+
65+ if response == None: # user cancelled
66+ raise StopIteration
67+ if response[0] == 0: # an issue about rhythmbox interface
68+ apport.hookutils.attach_gconf(report, 'rhythmbox')
69+ report['GConfNonDefault'] = mask_values(report['GConfNonDefault'])
70+ if response[0] == 1: # the issue is a sound one
71+ os.execlp('apport-bug', 'apport-bug', 'audio')
72+ if response[0] == 2: # the issue is a codec one
73+ report.add_package_info("libgstreamer0.10-0")
74+ return
75+ if response[0] == 3: # problem with the music store
76+ report.add_package_info("rhythmbox-ubuntuone-music-store")
77+ return
78+
79+ report["LogAlsaMixer"] = apport.hookutils.command_output(["/usr/bin/amixer"])
80+ report["GstreamerVersions"] = apport.hookutils.package_versions("gstreamer*")
81+ report["XorgLog"] = apport.hookutils.read_file("/var/log/Xorg.0.log")
82
83=== renamed file 'debian/source_rhythmbox.py' => 'debian/source_rhythmbox.py.moved'

Subscribers

People subscribed via source and target branches

to all changes: