Merge lp:~lool/ubuntu-download-manager/udm-no-debug into lp:ubuntu-download-manager

Proposed by Loïc Minier
Status: Merged
Approved by: Roberto Alsina
Approved revision: 151
Merged at revision: 151
Proposed branch: lp:~lool/ubuntu-download-manager/udm-no-debug
Merge into: lp:ubuntu-download-manager
Diff against target: 16 lines (+0/-4)
1 file modified
libubuntudownloadmanager/download_daemon.cpp (+0/-4)
To merge this branch: bzr merge lp:~lool/ubuntu-download-manager/udm-no-debug
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Roberto Alsina (community) Approve
Review via email: mp+191536@code.launchpad.net

Commit message

Always honor UBUNTU_DOWNLOADER_DEBUG; we should consider switching release builds to -UDEBUG, but this is too intrusive right now; LP: #1240656.

Description of the change

Always honor UBUNTU_DOWNLOADER_DEBUG; we should consider switching release builds to -UDEBUG, but this is too intrusive right now; LP: #1240656.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
Loïc Minier (lool) wrote :

Tested with click downloads; almost too quiet now; just logs:
2013-10-17 00:41:19,288 - WARNING - QObject::connect: No such signal SystemNetworkInfo::onlineStateChange(bool)
which is probably worth fixing too

We should tweak debug/trace messages

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'libubuntudownloadmanager/download_daemon.cpp'
2--- libubuntudownloadmanager/download_daemon.cpp 2013-09-20 15:54:50 +0000
3+++ libubuntudownloadmanager/download_daemon.cpp 2013-10-16 23:53:28 +0000
4@@ -158,12 +158,8 @@
5
6 // set logging
7 Logger::setupLogging();
8-#ifdef DEBUG
9- Logger::setLogLevel(QtDebugMsg);
10-#else
11 if (qgetenv("UBUNTU_DOWNLOADER_DEBUG") != "")
12 Logger::setLogLevel(QtDebugMsg);
13-#endif
14 }
15
16 private:

Subscribers

People subscribed via source and target branches