Comment 5 for bug 1036211

Revision history for this message
Bryce Harrington (bryce) wrote :

Yeah, sheer laziness on my part.

This bit of code is actually exceedingly unimportant. The tarball that is generated is purely for the user's use and is not uploaded anywhere or made use of in any form by xdiagnose. It was a community contribution from back in the days before we had a decent X apport hook. Now days we'd just have them file a bug via ubuntu-bug xorg. I suppose users might not have apport turned on or something, so this might conceivably still be of some use, however I see the function's begun to bitrot: it's looking for gdm rather than lightdm.

So, I think the right thing to do here is rather than fix the temp file, to just excise the functionality and focus on apport as the way to gather troubleshooting info.