Support package hooks in /opt

Bug #1020503 reported by Daniel Holbach
This bug affects 2 people
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Martin Pitt

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/<app>/etc/apport/crashdb.conf.d/<app>-crashdb.conf and
 - opt/<app>/share/apport/package-hooks/source_<app>.py

Tags: arb Edit Tag help
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/ 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:
    - Fix test for calling test runner with
      absolute path.
    - Fix crash on writing virtual_mapping.db when
      running with --sandbox-dir and -S system or giving no --cache.
    - 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.
    - Allow specifying a list of profile names when using
      attach_mac_events(). Thanks Marc Deslauriers.
    -, 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)
    -, 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
  * 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
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers