Apport

Merge lp://qastaging/~crimsun/apport/collect_alsa-kernel_version into lp://qastaging/~ubuntu-core-dev/ubuntu/karmic/apport/ubuntu

  1. collect_alsa-kernel_version
  2. Merge into ubuntu
Proposed by Daniel T Chen
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp://qastaging/~crimsun/apport/collect_alsa-kernel_version
Merge into: lp://qastaging/~ubuntu-core-dev/ubuntu/karmic/apport/ubuntu
Diff against target: 10 lines
1 file modified
apport/hookutils.py (+1/-0)
To merge this branch: bzr merge lp://qastaging/~crimsun/apport/collect_alsa-kernel_version
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+14268@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel T Chen (crimsun) wrote :

Ugh, that prior merge proposal ended up with the wrong radio button selected, sorry!

As summarised in bug #467233, we should collect the current alsa-kernel version for the running kernel. Doing so allows us to more easily recommend the installation of linux-backports-modules-alsa-karmic-generic, which has resolved numerous bugs on new IDT HDA codecs.

Revision history for this message
Martin Pitt (pitti) wrote :

I did an equivalent, but more robust commit into trunk (r1635). Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'apport/hookutils.py'
2--- apport/hookutils.py 2009-10-14 21:26:53 +0000
3+++ apport/hookutils.py 2009-10-31 19:30:25 +0000
4@@ -181,6 +181,7 @@
5 attach_file_if_exists(report, '/etc/asound.conf')
6
7 report['AlsaDevices'] = command_output(['ls','-l','/dev/snd/'])
8+ report['AlsaVersion'] = command_output(['cat','/proc/asound/version'])
9 report['AplayDevices'] = command_output(['aplay','-l'])
10 report['ArecordDevices'] = command_output(['arecord','-l'])
11

Subscribers

People subscribed via source and target branches