expose exceptions from hooks

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

Bug Description

Right now Apport only sends errors from hooks to stderr, where they are by and large invisible.

Instead, they should create a PackageHookException_hook_name field with the exception, so that we can process those in daisy, and also see them in LP bugs.

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

Fixed in trunk r2557.

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 - 2.7-0ubuntu1

---------------
apport (2.7-0ubuntu1) raring; urgency=low

  * New upstream release:
    - packaging.py, get_file_package(): Add optional "release" and "arch"
      arguments for getting a file's package for a foreign release or
      architecture. Implement this for apt/dpkg backend.
    - packaging.py, install_packages(): Add optional "architecture" argument
      for creating a sandbox for a foreign architecture. Implement this for
      apt/dpkg backend.
    - When a report's architecture does not match the system architecture, try
      to use gdb-multiarch (if available, as packaged on Debian/Ubuntu), and
      set architecture and gnutarget accordingly in gdb. This supports x86,
      x86_64, and ARM for now, so that reports from all these architectures
      can be retraced on an x86_84 machine. (LP: #1044437)
    - launchpad.py: Add "architecture" option to process reports for a foreign
      architecture.
    - Add exceptions from package hooks to new HookError_<filename> report
      field, to make them more visible. Until now they were only written to
      stderr. (LP: #1086309)
    - apport-cli: Fix showing of prompt. Thanks Dmitry Shachnev!
    - fileutils.py, mark_report_upload(): Do not try to remove the .uploaded
      file, as this is not owned by the user.
    - backends/packaging-apt-dpkg.py, install_packages(): Set mirror to the
      one in the sandbox config.
    - apportcheckresume: Fix crash if state file does not exist.
  * debian/control: Add "gdb-multiarch" suggests to apport-retrace, and update
    package description for justification.
 -- Martin Pitt <email address hidden> Mon, 10 Dec 2012 14:07:04 +0100

Changed in apport (Ubuntu):
status: Fix Committed → 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.