Mir

Merge lp:~alan-griffiths/mir/fix-1583536 into lp:mir

Proposed by Alan Griffiths
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 3515
Proposed branch: lp:~alan-griffiths/mir/fix-1583536
Merge into: lp:mir
Diff against target: 66 lines (+14/-2)
4 files modified
tests/mir_test/CMakeLists.txt (+4/-0)
tests/mir_test_doubles/CMakeLists.txt (+5/-1)
tests/mir_test_framework/CMakeLists.txt (+4/-0)
tests/mirtest.pc.in (+1/-1)
To merge this branch: bzr merge lp:~alan-griffiths/mir/fix-1583536
Reviewer Review Type Date Requested Status
Mir CI Bot continuous-integration Approve
Cemil Azizoglu (community) Approve
Kevin DuBois (community) Approve
Review via email: mp+295227@code.launchpad.net

Commit message

Make the mirtest-dev package useful for downstream projects

Description of the change

1. Don't compile the mir-test-assist objects for LTO
2. Include all the objects needed to link successfully
3. Include all the transitive dependencies in the .pc

To post a comment you must log in.
Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

FAILED: Continuous integration, rev:3512
https://mir-jenkins.ubuntu.com/job/mir-ci/997/
Executed test runs:
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-mir/1082/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1130
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1121
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1121
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1092/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1092
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1092/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1092
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1092/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1092
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1092/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1092
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1092/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/mir-ci/997/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

Failure is lp:1576690, triggering rebuild

Revision history for this message
Kevin DuBois (kdub) wrote :

lgtm

review: Approve
Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

FAILED: Continuous integration, rev:3512
https://mir-jenkins.ubuntu.com/job/mir-ci/999/
Executed test runs:
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-mir/1085/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1133
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1124
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1124
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1095
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1095/artifact/output/*zip*/output.zip
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1095/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1095
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1095/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1095
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1095/artifact/output/*zip*/output.zip
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1095/console

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/mir-ci/999/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

ok

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

> FAILED: Continuous integration, rev:3512
> https://mir-jenkins.ubuntu.com/job/mir-ci/999/
> Executed test runs:
> FAILURE: https://mir-jenkins.ubuntu.com/job/build-mir/1085/console
> SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1133
> SUCCESS: https://mir-
> jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1124
> SUCCESS: https://mir-
> jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1124
> SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-
> mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1095
> deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,
> compiler=clang,platform=mesa,release=vivid+overlay/1095/artifact/output/*zip*/
> output.zip
> FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-
> mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1095/console

lp:1576690

> SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-
> armhf,compiler=gcc,platform=android,release=vivid+overlay/1095
> deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-
> armhf,compiler=gcc,platform=android,release=vivid+overlay/1095/artifact/output
> /*zip*/output.zip
> SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-
> mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1095
> deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,c
> ompiler=gcc,platform=android,release=vivid+overlay/1095/artifact/output/*zip*/
> output.zip
> FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-
> mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1095/console

logged as
lp:1583970

> Click here to trigger a rebuild:
> https://mir-jenkins.ubuntu.com/job/mir-ci/999/rebuild

Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

PASSED: Continuous integration, rev:3512
https://mir-jenkins.ubuntu.com/job/mir-ci/1005/
Executed test runs:
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-mir/1095
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1143
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1134
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1134
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1105
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1105/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1105
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial/1105/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1105
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1105/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1105
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1105/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1105
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial/1105/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/mir-ci/1005/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/mir_test/CMakeLists.txt'
--- tests/mir_test/CMakeLists.txt 2015-06-23 16:08:35 +0000
+++ tests/mir_test/CMakeLists.txt 2016-05-19 15:39:22 +0000
@@ -1,3 +1,7 @@
1# We don't want LTO objects in a published archive (libmir-test-assist.a)
2string (REPLACE " -flto " " " CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
3string (REPLACE " -flto " " " CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
4
1add_library(mir-public-test OBJECT5add_library(mir-public-test OBJECT
2 cross_process_action.cpp6 cross_process_action.cpp
3 cross_process_sync.cpp7 cross_process_sync.cpp
48
=== modified file 'tests/mir_test_doubles/CMakeLists.txt'
--- tests/mir_test_doubles/CMakeLists.txt 2016-05-04 09:06:26 +0000
+++ tests/mir_test_doubles/CMakeLists.txt 2016-05-19 15:39:22 +0000
@@ -16,7 +16,6 @@
16 mock_input_device.cpp16 mock_input_device.cpp
17 mock_frame_dropping_policy_factory.cpp17 mock_frame_dropping_policy_factory.cpp
18 mock_timer.cpp18 mock_timer.cpp
19 stub_buffer.cpp
20 test_protobuf_client.cpp19 test_protobuf_client.cpp
21 test_protobuf_socket_server.cpp20 test_protobuf_socket_server.cpp
22 triggered_main_loop.cpp21 triggered_main_loop.cpp
@@ -71,11 +70,16 @@
71 )70 )
72endif()71endif()
7372
73# We don't want LTO objects in a published archive (libmir-test-assist.a)
74string (REPLACE " -flto " " " CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
75string (REPLACE " -flto " " " CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
76
74add_library(mir-public-test-doubles OBJECT77add_library(mir-public-test-doubles OBJECT
75 nested_mock_egl.cpp78 nested_mock_egl.cpp
76 null_logger.cpp79 null_logger.cpp
77 stub_display_configuration.cpp ${CMAKE_SOURCE_DIR}/include/test/mir/test/doubles/stub_display_configuration.h80 stub_display_configuration.cpp ${CMAKE_SOURCE_DIR}/include/test/mir/test/doubles/stub_display_configuration.h
78 fake_display.cpp ${CMAKE_SOURCE_DIR}/include/test/mir/test/doubles/fake_display.h81 fake_display.cpp ${CMAKE_SOURCE_DIR}/include/test/mir/test/doubles/fake_display.h
82 stub_buffer.cpp
79)83)
8084
81add_library(mir-test-doubles-static STATIC85add_library(mir-test-doubles-static STATIC
8286
=== modified file 'tests/mir_test_framework/CMakeLists.txt'
--- tests/mir_test_framework/CMakeLists.txt 2016-03-29 07:30:50 +0000
+++ tests/mir_test_framework/CMakeLists.txt 2016-05-19 15:39:22 +0000
@@ -21,6 +21,10 @@
21 -DMIR_BUILD_PREFIX="${CMAKE_BINARY_DIR}"21 -DMIR_BUILD_PREFIX="${CMAKE_BINARY_DIR}"
22 )22 )
2323
24# We don't want LTO objects in a published archive (libmir-test-assist.a)
25string (REPLACE " -flto " " " CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
26string (REPLACE " -flto " " " CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
27
24add_library(mir-public-test-framework OBJECT28add_library(mir-public-test-framework OBJECT
25 any_surface.cpp29 any_surface.cpp
26 async_server_runner.cpp30 async_server_runner.cpp
2731
=== modified file 'tests/mirtest.pc.in'
--- tests/mirtest.pc.in 2015-06-25 16:47:02 +0000
+++ tests/mirtest.pc.in 2016-05-19 15:39:22 +0000
@@ -6,5 +6,5 @@
6Description: Mir test assist library6Description: Mir test assist library
7Version: @MIR_VERSION@7Version: @MIR_VERSION@
8Requires: mirserver mirplatform mirclient8Requires: mirserver mirplatform mirclient
9Libs: -L@LIBDIR@ -lmir-test-assist9Libs: -L@LIBDIR@ -lmir-test-assist -ldl
10Cflags: -I@INCLUDEDIR@10Cflags: -I@INCLUDEDIR@

Subscribers

People subscribed via source and target branches