attach_root_command_outputs creates empty entries if no output from command
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
apport (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
See bug 813791 as an example.
Since the display manager log files are only readable by the superuser, we have to prompt for sudo password. Since there are several log files (and we want them attached as separate files to the bug report), to prevent prompting the user multiple times, we do this in one call:
'LightdmGreet
})
With the introduction of lightdm we now have two possible sets of log files, but it's likely the user will only have one set or the other. Unfortunately, in this case whichever set is missing will show up in the bug report as three empty fields.
As well, if a bug occurred on the user's first boot with gdm, they might not have :0.log.1 or .2.
attach_
Related branches
- Martin Pitt (community): Approve
-
Diff: 41 lines (+6/-4)1 file modifiedapport/hookutils.py (+6/-4)
This bug was fixed in the package apport - 1.21.3-0ubuntu1
---------------
apport (1.21.3-0ubuntu1) oneiric; urgency=low
* New upstream bug fix release. Changes since our previous snapshot: packaging. get_library_ paths() interface and implement it for packaging- apt-dpkg. py using dpkg multiarch directories. Use it hooks/ubuntu. py: Fix crash if report doesn't have a Package hooks/ubuntu. py: Fix crash if report doesn't have a
- Add apport.
backends/
in chroot.py. This unbreaks apport-chroot for oneiric.
- hookutils.py: Don't attach empty values. Thanks Bryce Harrington.
(LP: #813798)
- apport-gtk: Correctly pass message dialog type.
- apport-gtk: Fix GLib and GObject imports to be compatible with the future
pygobject 3.0.
- hookutils.py, attach_alsa(): Get a list of outputs/inputs that PulseAudio
knows about, which also shows the currently selected output/input, as well
as volumes. This should help with "no sound" bug troubleshooting. Thanks
Luke Yelavich.
* data/general-
field (caught by test suite).
* data/general-
DpkgTerminalLog field (caught by test suite).
-- Martin Pitt <email address hidden> Wed, 17 Aug 2011 08:42:18 +0200