Mir

Merge lp://qastaging/~vanvugt/mir/fix-1539338 into lp://qastaging/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://qastaging/~vanvugt/mir/fix-1539338
Merge into: lp://qastaging/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://qastaging/~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.qastaging.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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches