Mir

Merge lp:~vanvugt/mir/fix-1401365 into lp:mir

Proposed by Daniel van Vugt
Status: Merged
Merged at revision: 2146
Proposed branch: lp:~vanvugt/mir/fix-1401365
Merge into: lp:mir
Diff against target: 26 lines (+3/-3)
1 file modified
examples/CMakeLists.txt (+3/-3)
To merge this branch: bzr merge lp:~vanvugt/mir/fix-1401365
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+244377@code.launchpad.net

Commit message

Rename mir_demo_server back to mir_demo_server_basic.

It's still confusing given we also have mir_demo_server_{minimal,shell}
but this makes things more consistent at least, and fixes a CI build
failure (LP: #1401365)

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
Alan Griffiths (alan-griffiths) wrote :

Fixes the failure - we can argue about names at leisure

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'examples/CMakeLists.txt'
2--- examples/CMakeLists.txt 2014-12-11 02:43:01 +0000
3+++ examples/CMakeLists.txt 2014-12-11 04:09:07 +0000
4@@ -158,12 +158,12 @@
5
6 install(TARGETS ${INSTALL_DEMOS} RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
7
8-add_executable(mir_demo_server
9+add_executable(mir_demo_server_basic
10 server_example.cpp
11 glog_logger.cpp
12 )
13
14-target_link_libraries(mir_demo_server
15+target_link_libraries(mir_demo_server_basic
16 mirserver
17 exampleserverconfig
18 ${GLog_LIBRARY}
19@@ -171,7 +171,7 @@
20 ${Boost_LIBRARIES}
21 )
22
23-install(TARGETS mir_demo_server
24+install(TARGETS mir_demo_server_basic
25 RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
26 )
27

Subscribers

People subscribed via source and target branches