update alsa/audio related infomation collection code in apport

Bug #1893899 reported by Yuan-Chen Cheng
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OEM Priority Project
Fix Released
High
Yuan-Chen Cheng
apport (Ubuntu)
Fix Released
Undecided
Brian Murray
Groovy
Fix Released
Undecided
Brian Murray

Bug Description

what need to change

1. try to use /usr/sbin/alsa-info before try /usr/share/alsa-base/alsa-info.sh (weird for me that two copy of alsa-info co-exists, but anyway)
2. add pa-info (exist after focal)
3. add ucm2 directory. (exist after focal)

Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote : Re: update alsa/audio realted infomation collection code in apport
information type: Proprietary → Public
summary: - update alsa/audio realted infomation collection code
+ update alsa/audio realted infomation collection code in apport
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "apport-diff-groovy-audio-info-refinement.patch" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
tags: added: rls-gg-incoming
Balint Reczey (rbalint)
summary: - update alsa/audio realted infomation collection code in apport
+ update alsa/audio related infomation collection code in apport
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in apport (Ubuntu):
status: New → Confirmed
tags: removed: rls-gg-incoming
Revision history for this message
Brian Murray (brian-murray) wrote :

By adding in a call to '/usr/bin/pa-info' in apport/hookutils.py there ends up being duplicate information in the apport created report as '/usr/bin/pa-info' calls 'alsa-info' which was already included in the report. How do you feel about removing the calls to 'alsa-info' and just using 'pa-info'?

Thanks!

Changed in apport (Ubuntu Groovy):
status: Confirmed → Incomplete
assignee: nobody → Brian Murray (brian-murray)
Revision history for this message
Hui Wang (hui.wang) wrote :

the pa-info already includes calling the alsa-info.sh, it is ok to remove the alsa-info.

But it is better to keep the alsa-info, since the log of alsa-info is cleaner than the log of pa-info, pa-info includes many pulseaudio level logs. And If we want to exchange the log with upstream developers, we often send alsa-info.txt since it is possible that other Linux distributions don't use pulseaudio as the audio server, they only need a clean alsa-info.txt without pulseaudio logs.

Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

Is there a quick and clean way to extract alsa-info from output of pa-info?

Revision history for this message
Hui Wang (hui.wang) wrote :

Reply #7, no, I don't know how to extract alsa-info from output of pa-info.

Revision history for this message
Brian Murray (brian-murray) wrote :

I've gone ahead and uploaded this for Groovy but renamed the key pa-info to PaInfo so that it is consistent with the naming of other keys.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 2.20.11-0ubuntu47

---------------
apport (2.20.11-0ubuntu47) groovy; urgency=medium

  [ Tiago Stürmer Daitx ]
  * apport/ui.py: improve message when origin check fails as it can be
    caused by empty apt list - candidate is limited to dpkg and we can't
    tell where it came from. (LP: #1775219)

 -- Brian Murray <email address hidden> Mon, 21 Sep 2020 15:02:17 -0700

Changed in apport (Ubuntu Groovy):
status: Incomplete → Fix Released
tags: added: fr-692
Changed in oem-priority:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.