Mir

Merge lp:~mir-team/mir/package-arm64-drivers into lp:mir

Proposed by Daniel van Vugt
Status: Merged
Approved by: Kevin DuBois
Approved revision: no longer in the source branch.
Merged at revision: 3600
Proposed branch: lp:~mir-team/mir/package-arm64-drivers
Merge into: lp:mir
Diff against target: 72 lines (+8/-8)
2 files modified
debian/control (+7/-7)
debian/rules (+1/-1)
To merge this branch: bzr merge lp:~mir-team/mir/package-arm64-drivers
Reviewer Review Type Date Requested Status
Mir CI Bot continuous-integration Approve
Łukasz Zemczak Approve
Kevin DuBois (community) Approve
Review via email: mp+300438@code.launchpad.net

Commit message

Attempt to build and package missing arm64 binaries for
the Android platform (LP: #1579866)

Description of the change

Not actually tested on an arm64 system right now. I'm out of time today and can't think off hand how to test it.

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

FAILED: Continuous integration, rev:3597
https://mir-jenkins.ubuntu.com/job/mir-ci/1315/
Executed test runs:
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-mir/1542/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1595
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1586
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/1586
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/1586
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1557
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1557/artifact/output/*zip*/output.zip
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial+overlay/1557/console
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1557/console
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1557/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/1557
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1557/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/1557
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial+overlay/1557/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Kevin DuBois (kdub) wrote :

sure, lgtm.

Should probably be ran in a ppa builder though. (and would prefer if this landed after re-branching 0.24, if its not needed immediately)

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Builds fine at least - and that's all we need right now.

I did a test-build of the same changes just targeted for lp:mir/ubuntu in a silo (for testing) and arm64 builds succeed. Logs can be found here:

https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-004/+build/10487430

So a big +1 from me. Could we get this included in the nearest mir release? We want to build a xenial image this week.

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

@Kevin We would prefer it to be landed ASAP since it's blocking xenial rootfs builds.

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

sure, seems that we should do a 0.23.4 release. 0.24.0 has new features and will take longer to test, so landing independently in 0.23.4 will be the easiest way to go

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

FAILED: Continuous integration, rev:3597
https://mir-jenkins.ubuntu.com/job/mir-ci/1321/
Executed test runs:
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-mir/1553/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/1606
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1597
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/1597
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/1597
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1568
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=vivid+overlay/1568/artifact/output/*zip*/output.zip
    FAILURE: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial+overlay/1568/console
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1568
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=cross-armhf,compiler=gcc,platform=android,release=vivid+overlay/1568/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/1568
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/1568/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/1568
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial+overlay/1568/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Kevin DuBois (kdub) wrote :

CI failure was NestedServer failure, think we're okay to TA.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2016-07-15 02:11:27 +0000
3+++ debian/control 2016-07-19 09:25:35 +0000
4@@ -25,8 +25,8 @@
5 libprotobuf-dev,
6 pkg-config,
7 android-headers-23,
8- libhardware-dev [i386 amd64 armhf],
9- libandroid-properties-dev [i386 amd64 armhf],
10+ libhardware-dev [i386 amd64 armhf arm64],
11+ libandroid-properties-dev [i386 amd64 armhf arm64],
12 libgoogle-glog-dev,
13 liblttng-ust-dev,
14 libxkbcommon-dev,
15@@ -36,7 +36,7 @@
16 libgtest-dev,
17 google-mock (>= 1.6.0+svn437),
18 # only enable valgrind once it's been tested to work on each architecture:
19- valgrind [amd64 i386 armhf],
20+ valgrind [amd64 i386 armhf arm64],
21 libglib2.0-dev,
22 libfreetype6-dev,
23 abi-compliance-checker,
24@@ -277,7 +277,7 @@
25 Contains a tool for stress testing the Mir display server
26
27 Package: mir-android-diagnostics
28-Architecture: i386 amd64 armhf
29+Architecture: i386 amd64 armhf arm64
30 Pre-Depends: ${misc:Pre-Depends}
31 Depends: ${misc:Depends},
32 ${shlibs:Depends},
33@@ -332,7 +332,7 @@
34
35 Package: mir-platform-graphics-android10
36 Section: libs
37-Architecture: i386 amd64 armhf
38+Architecture: i386 amd64 armhf arm64
39 Multi-Arch: same
40 Pre-Depends: ${misc:Pre-Depends}
41 Depends: ${misc:Depends},
42@@ -388,7 +388,7 @@
43
44 Package: mir-client-platform-android5
45 Section: libs
46-Architecture: i386 amd64 armhf
47+Architecture: i386 amd64 armhf arm64
48 Multi-Arch: same
49 Pre-Depends: ${misc:Pre-Depends}
50 Depends: ${misc:Depends},
51@@ -419,7 +419,7 @@
52
53 Package: mir-graphics-drivers-android
54 Section: libs
55-Architecture: i386 amd64 armhf
56+Architecture: i386 amd64 armhf arm64
57 Multi-Arch: same
58 Pre-Depends: ${misc:Pre-Depends}
59 Depends: ${misc:Depends},
60
61=== modified file 'debian/rules'
62--- debian/rules 2016-05-03 06:55:25 +0000
63+++ debian/rules 2016-07-19 09:25:35 +0000
64@@ -29,7 +29,7 @@
65 endif
66
67 override_dh_auto_configure:
68-ifeq ($(DEB_HOST_ARCH),armhf)
69+ifneq ($(filter armhf arm64,$(DEB_HOST_ARCH)),)
70 dh_auto_configure -- \
71 $(COMMON_CONFIGURE_OPTIONS) \
72 -DMIR_RUN_ACCEPTANCE_TESTS=OFF \

Subscribers

People subscribed via source and target branches