symlinks create different python tracebacks

Bug #1179979 reported by Brian Murray
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Low
Martin Pitt
Precise
Won't Fix
Medium
Unassigned
Quantal
Won't Fix
Medium
Unassigned
Raring
Won't Fix
Medium
Unassigned

Bug Description

The package software-properties-common ships add-apt-repository and apt-add-repository (which is a symlink to add-apt-repository). Looking at errors.ubuntu.com I found two buckets with practically the same traceback although the binary name is different.

add-apt-repository: https://errors.ubuntu.com/problem/f5707ba3c48180238c0d96a61085173833011549

apt-add-repository: https://errors.ubuntu.com/problem/27b98aca2c8aa846cc5de86880f916adc56a68d3

It would be helpful if these had the same traceback so that they could be bucketed together.

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

I wrote a test case for this now. The actual ExecutableName field already gets resolved for symlinks, but the second occurrence in the duplicate signature actually comes from the exception itself. That's fairly tricky to resolve, but I'll think about it.

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

Fixed in trunk r2639.

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

---------------
apport (2.10.2-0ubuntu1) saucy; urgency=low

  * New upstream bug fix release:
    - Resolve symlinks in file references in Python crash signatures.
      (LP: #1179979)
    - Fix endless loop of EOFErrors with broken core dumps. (LP: #1168849)
    - report.py, add_gdb_info(): Fix crash if gdb did not determine an
      assertion message. (LP: #1171484)
    - apt/dpkg: Fix get_file_package()'s "release" field to actually mean
      DistroRelease:, not a distro code name. This now requires a previous
      call to install_packages() with a configdir, which needs to have a file
      <configdir>/<release>/codename with the release's code name in it.
    - sandboxutils.py: Call get_file_package() with the report's release, so
      that we actually get files from the target release, not the host
      release.
    - test_hookutils.py: Don't assume that /etc/motd exists.
  * Move to dh compat level 9.
 -- Martin Pitt <email address hidden> Wed, 22 May 2013 12:39:21 +0200

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
Changed in apport (Ubuntu Precise):
status: New → Triaged
Changed in apport (Ubuntu Quantal):
status: New → Triaged
Changed in apport (Ubuntu Raring):
status: New → Triaged
Changed in apport (Ubuntu Precise):
importance: Undecided → Medium
Changed in apport (Ubuntu Quantal):
importance: Undecided → Medium
Changed in apport (Ubuntu Raring):
importance: Undecided → Medium
Revision history for this message
Brian Murray (brian-murray) wrote :

Crash file about a symlink for which one should determine the crash signature.

Revision history for this message
Brian Murray (brian-murray) wrote :

Tool to return the crash signature of a crash file.

Revision history for this message
Brian Murray (brian-murray) wrote :

This only needs to be SRU'ed for daisy as it does the calculation of the crash signature.

Changed in apport (Ubuntu Quantal):
status: Triaged → Won't Fix
Changed in apport (Ubuntu Raring):
status: Triaged → Won't Fix
Revision history for this message
Martin Pitt (pitti) wrote :

Don't we use apport from trunk in daisy? I. e. on the server side?

Revision history for this message
Brian Murray (brian-murray) wrote :

Ah, right that is correct.

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.