would be nice to send xsessions-errors warnings and errors

Bug #431807 reported by Sebastien Bacher
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Wishlist
Martin Pitt

Bug Description

Binary package hint: apport

the regexp required is basically what pitti said on IRC: '^(.*:\d+): \w+\(WARNING|CRITICAL|ERROR\):.*'

Changed in apport (Ubuntu):
importance: Undecided → Wishlist
Changed in apport (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
status: New → Confirmed
Martin Pitt (pitti)
Changed in apport (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

trunk r1612

Changed in apport (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 1.9.2-0ubuntu1

---------------
apport (1.9.2-0ubuntu1) karmic; urgency=low

  * New upstream bug fix release:
    - apport-cli: Print the URL and ask whether to open a browser. In many
      situations (such as usage on a server through ssh), it's preferable to not
      open the browser on the reporting computer. Thanks to Matt Zimmerman for the
      initial patch! (LP: #286415)
    - general-hooks/generic.py: Collect important glib errors/assertions (which
      should not have private data) from ~/.xsession-errors (LP: #431807)
    - launchpad.py: Link hardware data submission key if it exists. (LP: #424382)
    - apport-cli: Fix crash with non-ASCII characters in prompts.
    - Fix "apport-bug symptomname" to actually work.
    - launchpad.py: Fix crash on invalid credentials file. Thanks to Marco
      Rodrigues for the initial patch! (LP: #414055)
  * man/apport-bug.1: Document APPORT_IGNORE_OBSOLETE_PACKAGES (cherrypicked
    from trunk), and APPORT_STAGING (Ubuntu specific change, since it's a
    launchpad backend specific variable).
  * bin/apport-bug: Ignore -p option when giving two arguments, to keep
    compatibility with current bug filing instructions. (LP: #356755)
  * debian/copyright: Update copyright year and fix GPL link, thanks Marco
    Rodrigues!

 -- Martin Pitt <email address hidden> Fri, 02 Oct 2009 13:25:58 +0200

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Martin von Gagern (gagern) wrote :

I came to this report here from bug #430705. I had the initial impression that this thing here was intended to provide a solution for that one. However, at least in its current form, it does not:

The errors I am concerned with are generated by the xkbcomp binary, and don't match the glib pattern quoted above.
I just did update to Karmic Beta, and confirmed that there is no magic wrapping these lines into glib error lines.

Unfortunately I can think of no way to easily recognize the xkbcomp errors as such. So properly extracting them from the concatenated log is probably impossible, and bug #328980 a cleaner solution in the long run.

For the specific error message I have in mind, however, you could probably add a suitable regexp:
'^Error: *No Symbols named'
This at least handles the case I encountered in bug #328980 and will help diagnose similar errors.
To be more generic, one could turn all occurrences of the "ERROR3" macro in the xkbcomp sources into patterns, but I don't know if that would be worth the effort.

Would you consider adding the regexp above, or is there a reason not to?

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

Sure, I added "^Error: .*No Symbols named" to the regexp in trunk r1619.

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.