Mir

Merge lp:~robert-ancell/mir/mirplatform into lp:~mir-team/mir/trunk

Proposed by Robert Ancell on 2013-08-07
Status: Rejected
Rejected by: Alan Griffiths on 2013-08-07
Proposed branch: lp:~robert-ancell/mir/mirplatform
Merge into: lp:~mir-team/mir/trunk
Diff against target: 29 lines (+3/-1)
2 files modified
src/server/CMakeLists.txt (+2/-0)
src/server/mirserver.pc.in (+1/-1)
To merge this branch: bzr merge lp:~robert-ancell/mir/mirplatform
Reviewer Review Type Date Requested Status
Alan Griffiths 2013-08-07 Needs Fixing on 2013-08-07
PS Jenkins bot (community) continuous-integration Needs Fixing on 2013-08-07
Review via email: mp+178895@code.launchpad.net

Commit message

Install and use mirplatform headers

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Alan Griffiths (alan-griffiths) wrote :

Looks like infrastructure - rebuilding

review: Approve
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Alan Griffiths (alan-griffiths) wrote :

CMake Error at src/server/CMakeLists.txt:77 (install):
  install DIRECTORY given unknown argument "/mir/include/server/mir".

(I'll submit a fixed version as Robert is offline)

review: Needs Fixing
Alan Griffiths (alan-griffiths) wrote :

> CMake Error at src/server/CMakeLists.txt:77 (install):
> install DIRECTORY given unknown argument "/mir/include/server/mir".
>
> (I'll submit a fixed version as Robert is offline)

https://code.launchpad.net/~alan-griffiths/mir/mirplatform-install-headers/+merge/178909

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/server/CMakeLists.txt'
2--- src/server/CMakeLists.txt 2013-08-06 07:20:02 +0000
3+++ src/server/CMakeLists.txt 2013-08-07 07:23:25 +0000
4@@ -17,6 +17,7 @@
5 set(EXEC_PREFIX "${CMAKE_INSTALL_PREFIX}")
6 set(LIBDIR "${CMAKE_INSTALL_FULL_LIBDIR}")
7 set(INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/mirserver")
8+set(PLATFORM_INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/mirplatform")
9
10 configure_file(
11 ${CMAKE_CURRENT_SOURCE_DIR}/mirserver.pc.in
12@@ -74,6 +75,7 @@
13 LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
14 )
15 install(DIRECTORY
16+ ${CMAKE_SOURCE_DIR}/include/platform/mir DESTINATION "include/mirplatform"
17 ${CMAKE_SOURCE_DIR}/include/server/mir DESTINATION "include/mirserver"
18 )
19 else()
20
21=== modified file 'src/server/mirserver.pc.in'
22--- src/server/mirserver.pc.in 2013-04-24 05:22:20 +0000
23+++ src/server/mirserver.pc.in 2013-08-07 07:23:25 +0000
24@@ -9,4 +9,4 @@
25 Requires.private: protobuf >= 2.4.1
26 Requires: mircommon
27 Libs: -L@LIBDIR@ -lmirserver
28-Cflags: -I@INCLUDEDIR@
29+Cflags: -I@PLATFORM_INCLUDEDIR@ -I@INCLUDEDIR@

Subscribers

People subscribed via source and target branches