Mir

Merge lp:~vanvugt/mir/reduce-scope-of-3p-sources into lp:~mir-team/mir/trunk

Proposed by Daniel van Vugt
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 496
Proposed branch: lp:~vanvugt/mir/reduce-scope-of-3p-sources
Merge into: lp:~mir-team/mir/trunk
Diff against target: 35 lines (+0/-3)
3 files modified
3rd_party/CMakeLists.txt (+0/-1)
CMakeLists.txt (+0/-1)
src/client/CMakeLists.txt (+0/-1)
To merge this branch: bzr merge lp:~vanvugt/mir/reduce-scope-of-3p-sources
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
Alexandros Frantzis (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+153074@code.launchpad.net

Commit message

Don't export MIR_3RD_PARTY_SOURCES globally. It's not needed. Even where it
was being used in mirclient was unnecessary because mirclient already
links to 3rd_party.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:494
http://jenkins.qa.ubuntu.com/job/mir-ci/67/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-quantal-amd64-ci/68//console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/mir-ci/67//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
Alexandros Frantzis (afrantzis) wrote :

Looks good.

review: Approve
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

Yes. Was daft.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file '3rd_party/CMakeLists.txt'
--- 3rd_party/CMakeLists.txt 2013-03-07 08:04:05 +0000
+++ 3rd_party/CMakeLists.txt 2013-03-13 07:55:23 +0000
@@ -17,7 +17,6 @@
17)17)
1818
19set(MIR_3RD_PARTY_INCLUDE_DIRECTORIES ${MIR_3RD_PARTY_INCLUDE_DIRECTORIES} PARENT_SCOPE)19set(MIR_3RD_PARTY_INCLUDE_DIRECTORIES ${MIR_3RD_PARTY_INCLUDE_DIRECTORIES} PARENT_SCOPE)
20SET(MIR_3RD_PARTY_SOURCES ${MIR_3RD_PARTY_SOURCES} PARENT_SCOPE)
21set(MIR_INPUT_ANDROID_COMPILE_FLAGS ${MIR_INPUT_ANDROID_COMPILE_FLAGS}20set(MIR_INPUT_ANDROID_COMPILE_FLAGS ${MIR_INPUT_ANDROID_COMPILE_FLAGS}
22 PARENT_SCOPE)21 PARENT_SCOPE)
2322
2423
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2013-03-13 07:41:46 +0000
+++ CMakeLists.txt 2013-03-13 07:55:23 +0000
@@ -155,7 +155,6 @@
155set(MIR_ANDROID_INCLUDE_DIRECTORIES) # to be filled by android-input155set(MIR_ANDROID_INCLUDE_DIRECTORIES) # to be filled by android-input
156set(MIR_ANDROID_INPUT_COMPILE_FLAGS) # to be filled by android-input156set(MIR_ANDROID_INPUT_COMPILE_FLAGS) # to be filled by android-input
157set(MIR_3RD_PARTY_INCLUDE_DIRECTORIES)157set(MIR_3RD_PARTY_INCLUDE_DIRECTORIES)
158set(MIR_3RD_PARTY_SOURCES)
159add_subdirectory(3rd_party/)158add_subdirectory(3rd_party/)
160include_directories(${MIR_3RD_PARTY_INCLUDE_DIRECTORIES})159include_directories(${MIR_3RD_PARTY_INCLUDE_DIRECTORIES})
161160
162161
=== modified file 'src/client/CMakeLists.txt'
--- src/client/CMakeLists.txt 2013-03-13 07:41:46 +0000
+++ src/client/CMakeLists.txt 2013-03-13 07:55:23 +0000
@@ -78,7 +78,6 @@
78 mirclient SHARED78 mirclient SHARED
7979
80 ${CLIENT_SOURCES}80 ${CLIENT_SOURCES}
81 ${MIR_3RD_PARTY_SOURCES}
82)81)
8382
84set_target_properties(83set_target_properties(

Subscribers

People subscribed via source and target branches