Mir

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

Proposed by Robert Ancell
Status: Rejected
Rejected by: Alan Griffiths
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 Needs Fixing
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+178895@code.launchpad.net

Commit message

Install and use mirplatform headers

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 :

Looks like infrastructure - rebuilding

review: Approve
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 :

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
Revision history for this message
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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/server/CMakeLists.txt'
--- src/server/CMakeLists.txt 2013-08-06 07:20:02 +0000
+++ src/server/CMakeLists.txt 2013-08-07 07:23:25 +0000
@@ -17,6 +17,7 @@
17set(EXEC_PREFIX "${CMAKE_INSTALL_PREFIX}")17set(EXEC_PREFIX "${CMAKE_INSTALL_PREFIX}")
18set(LIBDIR "${CMAKE_INSTALL_FULL_LIBDIR}")18set(LIBDIR "${CMAKE_INSTALL_FULL_LIBDIR}")
19set(INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/mirserver")19set(INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/mirserver")
20set(PLATFORM_INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/mirplatform")
2021
21configure_file(22configure_file(
22 ${CMAKE_CURRENT_SOURCE_DIR}/mirserver.pc.in23 ${CMAKE_CURRENT_SOURCE_DIR}/mirserver.pc.in
@@ -74,6 +75,7 @@
74 LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}75 LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
75 )76 )
76 install(DIRECTORY77 install(DIRECTORY
78 ${CMAKE_SOURCE_DIR}/include/platform/mir DESTINATION "include/mirplatform"
77 ${CMAKE_SOURCE_DIR}/include/server/mir DESTINATION "include/mirserver"79 ${CMAKE_SOURCE_DIR}/include/server/mir DESTINATION "include/mirserver"
78 )80 )
79else()81else()
8082
=== modified file 'src/server/mirserver.pc.in'
--- src/server/mirserver.pc.in 2013-04-24 05:22:20 +0000
+++ src/server/mirserver.pc.in 2013-08-07 07:23:25 +0000
@@ -9,4 +9,4 @@
9Requires.private: protobuf >= 2.4.19Requires.private: protobuf >= 2.4.1
10Requires: mircommon10Requires: mircommon
11Libs: -L@LIBDIR@ -lmirserver11Libs: -L@LIBDIR@ -lmirserver
12Cflags: -I@INCLUDEDIR@12Cflags: -I@PLATFORM_INCLUDEDIR@ -I@INCLUDEDIR@

Subscribers

People subscribed via source and target branches