Merge lp:~brian-murray/aptdaemon/package-version-in-crash into lp:aptdaemon

Proposed by Brian Murray
Status: Merged
Merged at revision: 968
Proposed branch: lp:~brian-murray/aptdaemon/package-version-in-crash
Merge into: lp:aptdaemon
Diff against target: 31 lines (+9/-1)
2 files modified
aptdaemon/crash.py (+7/-1)
aptdaemon/gtk3widgets.py (+2/-0)
To merge this branch: bzr merge lp:~brian-murray/aptdaemon/package-version-in-crash
Reviewer Review Type Date Requested Status
Michael Vogt Approve
Review via email: mp+221507@code.launchpad.net

Description of the change

See the related bug.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

This looks good, thanks. The change in gtk3widgets is unreleated I will exclude that.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'aptdaemon/crash.py'
--- aptdaemon/crash.py 2012-12-29 15:19:44 +0000
+++ aptdaemon/crash.py 2014-05-30 09:30:33 +0000
@@ -43,7 +43,13 @@
43 uid = 043 uid = 0
44 report = apport.Report("Crash")44 report = apport.Report("Crash")
45 report["Title"] = error45 report["Title"] = error
46 report["Package"] = "aptdaemon"46 package = "aptdaemon"
47 try:
48 package_version = apport.packaging.get_version(package)
49 except ValueError as e:
50 if 'does not exist' in e.message:
51 package_version = 'unknown'
52 report['Package'] = '%s %s' % (package, package_version)
47 report["SourcePackage"] = "aptdaemon"53 report["SourcePackage"] = "aptdaemon"
48 report["Traceback"] = traceback54 report["Traceback"] = traceback
49 report["ExecutablePath"] = "/usr/sbin/aptd"55 report["ExecutablePath"] = "/usr/sbin/aptd"
5056
=== modified file 'aptdaemon/gtk3widgets.py'
--- aptdaemon/gtk3widgets.py 2013-12-03 10:16:52 +0000
+++ aptdaemon/gtk3widgets.py 2014-05-30 09:30:33 +0000
@@ -520,6 +520,8 @@
520 text += get_download_status_from_enum(status)520 text += get_download_status_from_enum(status)
521 text += "</small>"521 text += "</small>"
522 model = self.get_model()522 model = self.get_model()
523 if not model:
524 return
523 try:525 try:
524 iter = self._download_map[uri]526 iter = self._download_map[uri]
525 except KeyError:527 except KeyError:

Subscribers

People subscribed via source and target branches

to status/vote changes: