Merge lp:~xnox/ubuntu/raring/apport/ubuntu into lp:~ubuntu-core-dev/ubuntu/raring/apport/ubuntu

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Martin Pitt
Approved revision: 2165
Merged at revision: 2168
Proposed branch: lp:~xnox/ubuntu/raring/apport/ubuntu
Merge into: lp:~ubuntu-core-dev/ubuntu/raring/apport/ubuntu
Diff against target: 46 lines (+16/-5)
2 files modified
data/package-hooks/source_ubiquity.py (+9/-5)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~xnox/ubuntu/raring/apport/ubuntu
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+153224@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

LGTM, please go ahead.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/package-hooks/source_ubiquity.py'
2--- data/package-hooks/source_ubiquity.py 2013-02-19 00:39:55 +0000
3+++ data/package-hooks/source_ubiquity.py 2013-03-13 18:55:30 +0000
4@@ -10,11 +10,14 @@
5
6
7 def add_installation_log(report, ident, name):
8- if os.path.exists('/var/log/installer/%s' % name):
9- f = '/var/log/installer/%s' % name
10- elif os.path.exists('/var/log/%s' % name):
11- f = '/var/log/%s' % name
12- else:
13+ f = False
14+ for try_location in ('/var/log/installer/%s',
15+ '/var/log/%s',
16+ '/var/log/upstart/%s'):
17+ if os.path.exists(try_location % name):
18+ f = try_location % name
19+ break
20+ if not f:
21 return
22
23 if os.access(f, os.R_OK):
24@@ -119,6 +122,7 @@
25 if response:
26 add_installation_log(report, 'UbiquityDebug', 'debug')
27 add_installation_log(report, 'UbiquityDm', 'dm')
28+ add_installation_log(report, 'UpstartUbiquity', 'ubiquity.log')
29 add_installation_log(report, 'Casper', 'casper.log')
30 add_installation_log(report, 'OemConfigLog', 'oem-config.log')
31 if 'OemConfigLog' in report:
32
33=== modified file 'debian/changelog'
34--- debian/changelog 2013-03-07 15:04:39 +0000
35+++ debian/changelog 2013-03-13 18:55:30 +0000
36@@ -1,3 +1,10 @@
37+apport (2.9.1-0ubuntu2) UNRELEASED; urgency=low
38+
39+ * Include ubiquity.log generated by upstart for the ubiquity.conf
40+ upstart job.
41+
42+ -- Dmitrijs Ledkovs <dmitrij.ledkov@ubuntu.com> Wed, 13 Mar 2013 18:52:21 +0000
43+
44 apport (2.9.1-0ubuntu1) raring; urgency=low
45
46 * New upstream release:

Subscribers

People subscribed via source and target branches