Mir

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

Proposed by Daniel van Vugt
Status: Merged
Approved by: Brandon Schaefer
Approved revision: no longer in the source branch.
Merged at revision: 3713
Proposed branch: lp:~vanvugt/mir/fix-1626866
Merge into: lp:mir
Diff against target: 41 lines (+5/-2)
3 files modified
include/platform/mir/graphics/gamma_curves.h (+5/-0)
src/platforms/android/server/CMakeLists.txt (+0/-1)
src/platforms/mesa/CMakeLists.txt (+0/-1)
To merge this branch: bzr merge lp:~vanvugt/mir/fix-1626866
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Mir CI Bot continuous-integration Approve
Review via email: mp+306585@code.launchpad.net

Commit message

Fix a couple of recent mistakes that are making 'make abi-check' fail
to build:
  * Missing header guard (LP: #1626866)
  * Include paths no longer exist (they moved but seemingly also unused)

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

PASSED: Continuous integration, rev:3714
https://mir-jenkins.ubuntu.com/job/mir-ci/1800/
Executed test runs:
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-mir/2250
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/2313
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2304
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2304
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2304
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=yakkety/2278
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=yakkety/2278/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial+overlay/2278
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial+overlay/2278/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=yakkety/2278
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=yakkety/2278/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/2278
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/2278/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial+overlay/2278
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial+overlay/2278/artifact/output/*zip*/output.zip

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

review: Approve (continuous-integration)
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Opps lgtm, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'include/platform/mir/graphics/gamma_curves.h'
--- include/platform/mir/graphics/gamma_curves.h 2016-09-19 04:16:15 +0000
+++ include/platform/mir/graphics/gamma_curves.h 2016-09-23 07:45:46 +0000
@@ -16,6 +16,9 @@
16 * Authored by: Brandon Schaefer <brandon.schaefer@canonical.com>16 * Authored by: Brandon Schaefer <brandon.schaefer@canonical.com>
17 */17 */
1818
19#ifndef MIR_GRAPHICS_GAMMA_CURVES_H_
20#define MIR_GRAPHICS_GAMMA_CURVES_H_
21
19#include <cstdint>22#include <cstdint>
20#include <vector>23#include <vector>
2124
@@ -40,3 +43,5 @@
4043
41}44}
42}45}
46
47#endif
4348
=== modified file 'src/platforms/android/server/CMakeLists.txt'
--- src/platforms/android/server/CMakeLists.txt 2016-08-16 19:34:28 +0000
+++ src/platforms/android/server/CMakeLists.txt 2016-09-23 07:45:46 +0000
@@ -1,6 +1,5 @@
1include_directories(1include_directories(
2 ${PROJECT_SOURCE_DIR}/src/include/gl2 ${PROJECT_SOURCE_DIR}/src/include/gl
3 ${PROJECT_SOURCE_DIR}/src/include/sw
4)3)
54
6include_directories(5include_directories(
76
=== modified file 'src/platforms/mesa/CMakeLists.txt'
--- src/platforms/mesa/CMakeLists.txt 2016-08-02 18:30:00 +0000
+++ src/platforms/mesa/CMakeLists.txt 2016-09-23 07:45:46 +0000
@@ -8,7 +8,6 @@
88
9include_directories(9include_directories(
10 ${CMAKE_SOURCE_DIR}/include/platforms/mesa10 ${CMAKE_SOURCE_DIR}/include/platforms/mesa
11 ${PROJECT_SOURCE_DIR}/src/platforms/common/kms-utils/
12 ${PROJECT_SOURCE_DIR}/src/platforms/mesa/include/11 ${PROJECT_SOURCE_DIR}/src/platforms/mesa/include/
13)12)
1413

Subscribers

People subscribed via source and target branches