Merge lp:~bladernr/checkbox/949745 into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Merged at revision: 1312
Proposed branch: lp:~bladernr/checkbox/949745
Merge into: lp:checkbox
Diff against target: 28 lines (+6/-2)
2 files modified
checkbox/application.py (+1/-1)
debian/changelog (+5/-1)
To merge this branch: bzr merge lp:~bladernr/checkbox/949745
Reviewer Review Type Date Requested Status
Marc Tardif (community) Approve
Review via email: mp+96583@code.launchpad.net

Description of the change

Per ara's bug, set the default log level for checkbox down from debug to info. this should only apply to people running checkbox for UF or other purposes as the various checkbox overlays set this in their config options individually...

To post a comment you must log in.
Revision history for this message
Marc Tardif (cr3) wrote :

Looks perfectly reasonable, this might improve performance too!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox/application.py'
2--- checkbox/application.py 2010-09-23 10:11:27 +0000
3+++ checkbox/application.py 2012-03-08 14:34:19 +0000
4@@ -60,7 +60,7 @@
5 application_factory = Application
6
7 default_log = os.path.join(get_variable("CHECKBOX_DATA", "."), "checkbox.log")
8- default_log_level = "debug"
9+ default_log_level = "info"
10
11 def parse_options(self, args):
12 usage = _("Usage: checkbox [OPTIONS]")
13
14=== modified file 'debian/changelog'
15--- debian/changelog 2012-03-08 12:19:26 +0000
16+++ debian/changelog 2012-03-08 14:34:19 +0000
17@@ -59,7 +59,11 @@
18 * Modifications to removable_storage_test to handle cases where removable
19 media is not mounted prior to test running. (LP: #944623)
20
21- -- Jeff Lane <jeff@ubuntu.com> Mon, 05 Mar 2012 13:43:02 -0500
22+ [Jeff Lane]
23+ * Reset default checkbox log level to INFO from DEBUG to make logs less
24+ confusing and verbose. (LP: #949745)
25+
26+ -- Jeff Lane <jeff@ubuntu.com> Thu, 08 Mar 2012 09:29:10 -0500
27
28 checkbox (0.13.3) precise; urgency=low
29

Subscribers

People subscribed via source and target branches