Mir

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

Proposed by Daniel van Vugt
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 3280
Proposed branch: lp:~vanvugt/mir/fix-1539338
Merge into: lp:mir
Diff against target: 24 lines (+2/-2)
2 files modified
cmake/FindLibHardware.cmake (+1/-1)
src/platforms/android/common/syncfence.cpp (+1/-1)
To merge this branch: bzr merge lp:~vanvugt/mir/fix-1539338
Reviewer Review Type Date Requested Status
Kevin DuBois (community) Approve
Alan Griffiths Approve
Andreas Pokorny (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Mir CI Bot continuous-integration Approve
Review via email: mp+284389@code.launchpad.net

Commit message

Workaround/fix broken xenial build, due to last night's changes to the
android-headers packages. (LP: #1539338)

They moved all the Android headers we love from:
   /usr/include/android/
to:
   /usr/include/android-19/
leaving the former empty. And to use the latter you have to request a
different pkgconfig module (which we do here).

Description of the change

I have tried to make this change theoretically compatible with wily and vivid. Although that's not yet true, due to:
  * Our cross-compile scripts seem to prefer the host headers over the chroot headers; and
  * New identifiers recently added in libinput that we use, but don't exist on wily.

So compilation on wily/vivid support will be fixed later, but is not made any worse by this branch :)

This branch is also designed to be safe to keep even if the android-headers in Ubuntu archive gets reverted or fixed.

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

PASSED: Continuous integration, rev:3276
https://mir-jenkins.ubuntu.com/job/mir-ci/185/
Executed test runs:
    None: https://mir-jenkins.ubuntu.com/job/generic-update-mp/185/console

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

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3276
http://jenkins.qa.ubuntu.com/job/mir-ci/6159/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-android-vivid-i386-build/5733
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-clang-vivid-amd64-build/4640
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-vivid-touch/5689
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-xenial-touch/359
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-xenial-amd64-ci/483
        deb: http://jenkins.qa.ubuntu.com/job/mir-xenial-amd64-ci/483/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-xenial-i386-ci/483
        deb: http://jenkins.qa.ubuntu.com/job/mir-xenial-i386-ci/483/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/5686
        deb: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/5686/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/8113
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27158
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-xenial-armhf/355
        deb: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-xenial-armhf/355/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-xenial-touch/211
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27162

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/mir-ci/6159/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

lgtm

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

Harmless

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

immediate problem fixed, perhaps, but still some deeper digging needed into our dependencies to sort this out generically.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmake/FindLibHardware.cmake'
2--- cmake/FindLibHardware.cmake 2014-03-06 06:05:17 +0000
3+++ cmake/FindLibHardware.cmake 2016-01-29 04:09:27 +0000
4@@ -7,7 +7,7 @@
5 INCLUDE(FindPackageHandleStandardArgs)
6
7 find_package( PkgConfig )
8-pkg_check_modules(ANDROID_HEADERS REQUIRED android-headers)
9+pkg_search_module(ANDROID_HEADERS REQUIRED android-headers-19 android-headers)
10 set(LIBHARDWARE_INCLUDE_DIRS ${ANDROID_HEADERS_INCLUDE_DIRS})
11
12 find_library(LIBHARDWARE_LIBRARY
13
14=== modified file 'src/platforms/android/common/syncfence.cpp'
15--- src/platforms/android/common/syncfence.cpp 2016-01-20 23:59:18 +0000
16+++ src/platforms/android/common/syncfence.cpp 2016-01-29 04:09:27 +0000
17@@ -20,7 +20,7 @@
18
19 #include <unistd.h>
20 #include <sys/ioctl.h>
21-#include <android/linux/sync.h>
22+#include <linux/sync.h> // This will come from /usr/include/android{,-19}/
23
24 namespace mga = mir::graphics::android;
25

Subscribers

People subscribed via source and target branches