Merge lp:~brian-murray/aptdaemon/natty-check-if-apport-enabled into lp:~aptdaemon-developers/aptdaemon/ubuntu-natty

Proposed by Brian Murray
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp:~brian-murray/aptdaemon/natty-check-if-apport-enabled
Merge into: lp:~aptdaemon-developers/aptdaemon/ubuntu-natty
Diff against target: 38 lines (+19/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/check_if_apport_enabled.patch (+11/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~brian-murray/aptdaemon/natty-check-if-apport-enabled
Reviewer Review Type Date Requested Status
Martin Pitt (community) Needs Fixing
Michael Vogt Pending
Review via email: mp+64728@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Note: Please don't merge this patch, but the one that got committed to trunk: http://bazaar.launchpad.net/~aptdaemon-developers/aptdaemon/main/revision/661

Also, we don't actually need a debian/patches/* thing, a new bzr trunk snapshot should do.

Revision history for this message
Martin Pitt (pitti) :
review: Needs Fixing
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks Martin! I did a bzr snapshot upload now into natty proposed.

Revision history for this message
Michael Vogt (mvo) wrote :

I mark the branch merged (its merged in a slightly different way).

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-05-16 09:45:59 +0000
3+++ debian/changelog 2011-06-15 18:19:33 +0000
4@@ -1,3 +1,10 @@
5+aptdaemon (0.41+bzr657-0ubuntu1.2) natty-proposed; urgency=low
6+
7+ * patches/check_if_apport_enabled.patch:
8+ - only write crash if apport is enabled (LP: #797847)
9+
10+ -- Brian Murray <brian@ubuntu.com> Wed, 15 Jun 2011 11:02:46 -0700
11+
12 aptdaemon (0.41+bzr657-0ubuntu1.1) natty-proposed; urgency=low
13
14 * patches/fix_lp_781874.patch:
15
16=== added file 'debian/patches/check_if_apport_enabled.patch'
17--- debian/patches/check_if_apport_enabled.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/check_if_apport_enabled.patch 2011-06-15 18:19:33 +0000
19@@ -0,0 +1,11 @@
20+--- aptdaemon/crash.py 2011-01-25 04:46:52 +0000
21++++ aptdaemon/crash.py 2011-06-15 16:44:46 +0000
22+@@ -93,6 +93,7 @@
23+
24+ if __name__ == "__main__":
25+ apt_pkg.init_config()
26+- create_report()
27++ if apport.packaging.enabled():
28++ create_report()
29+
30+ # vim:ts=4:sw=4:et
31
32=== modified file 'debian/patches/series'
33--- debian/patches/series 2011-05-16 09:45:59 +0000
34+++ debian/patches/series 2011-06-15 18:19:33 +0000
35@@ -1,2 +1,3 @@
36 02_fix_crash_on_dpkg_disappear.patch
37 fix_lp_781874.patch
38+check_if_apport_enabled.patch

Subscribers

People subscribed via source and target branches