Merge lp:~stolowski/unity-scopes-api/scope-config-compile-fix into lp:unity-scopes-api

Proposed by Paweł Stołowski
Status: Merged
Approved by: Michi Henning
Approved revision: 336
Merged at revision: 338
Proposed branch: lp:~stolowski/unity-scopes-api/scope-config-compile-fix
Merge into: lp:unity-scopes-api
Diff against target: 10 lines (+1/-0)
1 file modified
src/scopes/internal/ScopeConfig.cpp (+1/-0)
To merge this branch: bzr merge lp:~stolowski/unity-scopes-api/scope-config-compile-fix
Reviewer Review Type Date Requested Status
James Henstridge Approve
Michi Henning (community) Approve
Review via email: mp+265097@code.launchpad.net

Commit message

Added missing #include needed for g++ 5.0.

Description of the change

Added missing #include needed for g++ 5.0. Proposing for trunk to get these quickfixes landed faster than via staging.

To post a comment you must log in.
Revision history for this message
Michi Henning (michihenning) wrote :

Yes, I've been bitten by this before too. It's a good idea to remember to compile in release mode every now and then. Of course, I rarely remember until *after* it's blown up on me in Jenkins :-(

review: Approve
Revision history for this message
James Henstridge (jamesh) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/scopes/internal/ScopeConfig.cpp'
--- src/scopes/internal/ScopeConfig.cpp 2015-06-11 11:11:42 +0000
+++ src/scopes/internal/ScopeConfig.cpp 2015-07-17 09:03:39 +0000
@@ -28,6 +28,7 @@
28#include <algorithm>28#include <algorithm>
29#include <sstream>29#include <sstream>
30#include <string>30#include <string>
31#include <iostream>
3132
32using namespace std;33using namespace std;
3334

Subscribers

People subscribed via source and target branches

to all changes: