Merge lp:~gerboland/unity-mir/enable-logs into lp:unity-mir

Proposed by Gerry Boland
Status: Work in progress
Proposed branch: lp:~gerboland/unity-mir/enable-logs
Merge into: lp:unity-mir
Diff against target: 9 lines (+2/-0)
1 file modified
debian/rules (+2/-0)
To merge this branch: bzr merge lp:~gerboland/unity-mir/enable-logs
Reviewer Review Type Date Requested Status
Gerry Boland (community) Needs Fixing
PS Jenkins bot (community) continuous-integration Needs Fixing
Michael Zanetti (community) Approve
Review via email: mp+222300@code.launchpad.net

Commit message

Enable all logging

Description of the change

Enable all logging

 * Are there any related MPs required for this MP to build/function as expected? Please list.
N
 * Did you perform an exploratory manual test run of your code change and any related functionality?
Y
 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Seems dpkg-buildpackage actually sets the build type and bypasses this. I think we want to do this:

debian/rules

override_dh_auto_configure:
        dh_auto_configure -- -DCMAKE_BUILD_TYPE=Debug

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?

yes. still works and prints debug stuff

 * Did CI run pass? If not, please explain why.

No. seems some disk error happening on jenkins...

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Gerry Boland (gerboland) wrote :

DASSERT now is ASSERT - breaks tests

review: Needs Fixing

Unmerged revisions

232. By Gerry Boland

idiot, tabls not spaces

231. By Gerry Boland

Override auto_configure to set cmake build type to debug

230. By Gerry Boland

Revert that

229. By Gerry Boland

Turn on all debug logging

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2014-01-09 09:04:50 +0000
3+++ debian/rules 2014-06-06 13:09:23 +0000
4@@ -6,3 +6,5 @@
5 %:
6 dh $@ --parallel --fail-missing
7
8+override_dh_auto_configure:
9+ dh_auto_configure -- -DCMAKE_BUILD_TYPE=Debug

Subscribers

People subscribed via source and target branches