Mir

Merge lp:~raof/mir/fix-g++-4.8-build into lp:~mir-team/mir/trunk

Proposed by Chris Halse Rogers
Status: Merged
Approved by: Robert Carr
Approved revision: no longer in the source branch.
Merged at revision: 683
Proposed branch: lp:~raof/mir/fix-g++-4.8-build
Merge into: lp:~mir-team/mir/trunk
Diff against target: 15 lines (+4/-1)
1 file modified
tests/CMakeLists.txt (+4/-1)
To merge this branch: bzr merge lp:~raof/mir/fix-g++-4.8-build
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Robert Carr (community) Approve
Review via email: mp+163635@code.launchpad.net

Commit message

Fix build with g++ 4.8

Description of the change

Fix the build with g++ 4.8

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Carr (robertcarr) wrote :

Ok!

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) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/CMakeLists.txt'
2--- tests/CMakeLists.txt 2013-05-10 00:39:29 +0000
3+++ tests/CMakeLists.txt 2013-05-14 05:47:28 +0000
4@@ -14,7 +14,10 @@
5 # Avoid clang complaints about poor quality gmock/gtest/cucumber headers
6 set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-error=null-dereference")
7 set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-error=overloaded-virtual")
8-endif ()
9+else ()
10+ # GCC 4.8 dislikes the gmock headers…
11+ set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-error=unused-local-typedefs")
12+endif()
13
14 include_directories(${PROJECT_SOURCE_DIR}/include/server)
15 include_directories(${PROJECT_SOURCE_DIR}/include/client)

Subscribers

People subscribed via source and target branches