Support package hooks in /opt

Bug #1020503 reported by Daniel Holbach
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Medium
Martin Pitt
Precise
Won't Fix
Undecided
Unassigned

Bug Description

As apps in extras.u.c are not allowed to ship to /usr, apport needs to learn to deal with files like
 - opt/extras.ubuntu.com/<app>/etc/apport/crashdb.conf.d/<app>-crashdb.conf and
 - opt/extras.ubuntu.com/<app>/share/apport/package-hooks/source_<app>.py

Tags: arb
Martin Pitt (pitti)
summary: - Learn to deal with /opt
+ Support package hooks and crash db configuration files in /opt
Changed in apport (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Martin Pitt (pitti)
Changed in apport (Ubuntu Precise):
status: New → Triaged
Revision history for this message
Martin Pitt (pitti) wrote : Re: Support package hooks and crash db configuration files in /opt

For crashdb configuration, I propose to generalize this so that package hooks can define a new crash db inline, instead of having to ship files in the first place. This is handled in bug 551330. So I devote this bug to finding package hooks in /opt.

summary: - Support package hooks and crash db configuration files in /opt
+ Support package hooks in /opt
Martin Pitt (pitti)
Changed in apport (Ubuntu):
status: Triaged → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

Fixed in trunk r2466. I manually tested with creating this:

$ cat /opt/extras.ubuntu.com/picsaw/share/apport/package-hooks/source_picsaw.pydef add_info(report, ui):
    report['CrashDB'] = '{"impl": "launchpad", "project": "picsaw"}'
    report['AAA'] = 'MOOO!'

$ apport-bug picsaw

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.5.1-0ubuntu1

---------------
apport (2.5.1-0ubuntu1) quantal; urgency=low

  * New upstream release:
    - test_recoverable_problem.py: Fix test for calling test runner with
      absolute path.
    - packaging-apt-dpkg.py: Fix crash on writing virtual_mapping.db when
      running with --sandbox-dir and -S system or giving no --cache.
    - REThread.py: Fix re-raising of exceptions in Python 3. Thanks Martin
      Packman! (LP: #1024836)
    - apport-retrace: Keep compressed CoreDump from .crash files instead of
      uncompressing them into memory. This dramatically reduces memory usage.
      (LP: #981155)
    - Add an apport.memdbg() function which prints out current memory usage if
      APPORT_MEMDEBUG is set. Annotate apport-retrace with it.
    - hookutils.py: Allow specifying a list of profile names when using
      attach_mac_events(). Thanks Marc Deslauriers.
    - hookutils.py, attach_root_command_outputs() and root_command_output():
      Drop usage of sudo, kdesudo, and gksu, and replace with pkexec. For
      attach_root_command_outputs(), use a wrapper and proper .policy file
      which explains the action and works under every environment; thus
      attach_root_command_outputs() is preferred over root_command_output()
      now, as it provides a better user experience.
    - Package hooks which want to send the report to a different crash
      database than "default" can now also give the database specification
      itself in the "CrashDB" field, not just the DB name. With this, packages
      do not need to ship a separate /etc/apport/crashdb.conf.d/ file. Please
      see doc/package-hooks.txt for details. (LP: #551330)
    - report.py, add_hooks_info(): If reporting against a package/program in
      /opt, also search for package hooks in the corresponding /opt directory.
      This allows such hooks to define a custom crash database and thus report
      bugs against their own project instead of against the distribution.
      (LP: #1020503)
  * debian/rules: Call dh_install with --list-missing, and ignore .pyc and
    .egg-info files.
  * debian/apport.install: Install new polkit .policy and root_info_wrapper
    files, and add policykit-1 Recommends.
  * debian/apport.install: Ship is-enabled script.
  * debian/apport-kde.install: Ship userpass.ui file.
  * Drop debian/apport-retrace.manpages and debian/apport.manpages and install
    the man pages from *.install instead. This works better with
    --list-missing.
  * debian/apport.install: Install 000record-status pm-utils script, to
    unbreak suspend failure detection.
 -- Martin Pitt <email address hidden> Wed, 22 Aug 2012 11:58:45 +0200

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Steve Langasek (vorlon) wrote :

The Precise Pangolin has reached end of life, so this bug will not be fixed for that release

Changed in apport (Ubuntu Precise):
status: Triaged → Won't Fix
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.