APT

Merge lp:~brian-murray/apt/mushed-files into lp:~ubuntu-core-dev/apt/ubuntu

Proposed by Brian Murray
Status: Merged
Merged at revision: 1887
Proposed branch: lp:~brian-murray/apt/mushed-files
Merge into: lp:~ubuntu-core-dev/apt/ubuntu
Diff against target: 24 lines (+4/-0)
2 files modified
apt-pkg/deb/dpkgpm.cc (+1/-0)
debian/changelog (+3/-0)
To merge this branch: bzr merge lp:~brian-murray/apt/mushed-files
Reviewer Review Type Date Requested Status
Ubuntu Core Development Team Pending
Review via email: mp+65756@code.launchpad.net

Description of the change

Bug 799127 is an example of many bugs where we can see AptOrdering stuck in DpkgTerminalLog.txt - https://launchpadlibrarian.net/73730089/DpkgTerminalLog.txt. I believe this happens because apport expects keys like 'AptOrdering:' to appear on a new line and AptOrdering is getting put on the same line as text in DpkgTerminalLog. Like so:

invoke-rc.d: initscript postgresql-8.4, action "start" failed.AptOrdering:

We can prevent this by adding in an extra new line after adding the contents of /var/log/apt/term.log as I've done.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'apt-pkg/deb/dpkgpm.cc'
2--- apt-pkg/deb/dpkgpm.cc 2011-06-15 21:02:51 +0000
3+++ apt-pkg/deb/dpkgpm.cc 2011-06-24 01:02:33 +0000
4@@ -1442,6 +1442,7 @@
5 {
6 while( fgets(buf, sizeof(buf), log) != NULL)
7 fprintf(report, " %s", buf);
8+ fprintf(report, " \n");
9 fclose(log);
10 }
11 }
12
13=== modified file 'debian/changelog'
14--- debian/changelog 2011-06-15 21:02:51 +0000
15+++ debian/changelog 2011-06-24 01:02:33 +0000
16@@ -8,6 +8,9 @@
17 * apt-pkg/deb/dpkgpm.cc:
18 - updated allocate memory string
19 - cannot access archive string is lowercase
20+ * apt-pkg/deb/dpkgpm.cc:
21+ - resolve issue where AptOrdering is included in DpkgTerminalLog in apport
22+ attachments
23
24 -- Brian Murray <brian@ubuntu.com> Wed, 15 Jun 2011 14:00:43 -0700
25

Subscribers

People subscribed via source and target branches

to all changes: