Merge lp:~cjwatson/launchpad-buildd/remove-qemu into lp:launchpad-buildd

Proposed by Colin Watson
Status: Merged
Merged at revision: 236
Proposed branch: lp:~cjwatson/launchpad-buildd/remove-qemu
Merge into: lp:launchpad-buildd
Diff against target: 77 lines (+3/-24)
4 files modified
bin/slave-prep (+1/-1)
bin/update-debian-chroot (+0/-22)
debian/changelog (+2/-0)
debian/control (+0/-1)
To merge this branch: bzr merge lp:~cjwatson/launchpad-buildd/remove-qemu
Reviewer Review Type Date Requested Status
William Grant Approve
Review via email: mp+328083@code.launchpad.net

Commit message

Drop qemu emulation support. It was quite unreliable, and we've had
real hardware for a while.

To post a comment you must log in.
234. By Colin Watson

Merge trunk.

235. By Colin Watson

Merge trunk.

236. By Colin Watson

Merge trunk.

Revision history for this message
William Grant (wgrant) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/slave-prep'
2--- bin/slave-prep 2017-07-25 22:11:19 +0000
3+++ bin/slave-prep 2017-07-28 11:12:54 +0000
4@@ -9,7 +9,7 @@
5
6 NTPDATE=ntpdate
7 SUDO=sudo
8-PACKAGES="launchpad-buildd python-lpbuildd sbuild bzr-builder bzr git-build-recipe git dpkg-dev python-debian qemu-user-static"
9+PACKAGES="launchpad-buildd python-lpbuildd sbuild bzr-builder bzr git-build-recipe git dpkg-dev python-debian"
10 KERNEL=$(uname -snrvm)
11
12 echo "Forking launchpad-buildd slave process..."
13
14=== modified file 'bin/update-debian-chroot'
15--- bin/update-debian-chroot 2017-07-25 22:11:19 +0000
16+++ bin/update-debian-chroot 2017-07-28 11:12:54 +0000
17@@ -23,8 +23,6 @@
18
19 echo "Updating debian chroot for build $BUILDID"
20
21-hostarch=$(dpkg --print-architecture)
22-
23 UNAME26=""
24 case $SUITE in
25 hardy*|lucid*|maverick*|natty*|oneiric*|precise*)
26@@ -43,26 +41,6 @@
27 ;;
28 esac
29
30-# Anything more fancy than amd64->ia32 is done with qemu+syscall
31-# emulation, handled via binfmt.misc hints to the kernel that require
32-# the statically-compiled qemu binary to appear in the same place both
33-# inside and outside the chroot.
34-if [ "$hostarch" != "$ARCHITECTURETAG" ]
35-then
36- case "$hostarch" in
37- i386|amd64|lpia)
38- case "$ARCHITECTURETAG" in
39- armel|armhf)
40- $SUDO /usr/bin/install /usr/bin/qemu-arm-static $ROOT/usr/bin/
41- ;;
42- arm64)
43- $SUDO /usr/bin/install /usr/bin/qemu-aarch64-static $ROOT/usr/bin/
44- ;;
45- esac
46- ;;
47- esac
48-fi
49-
50 export LANG=C
51 export DEBIAN_FRONTEND=noninteractive
52 export TTY=unknown
53
54=== modified file 'debian/changelog'
55--- debian/changelog 2017-07-28 10:12:15 +0000
56+++ debian/changelog 2017-07-28 11:12:54 +0000
57@@ -4,6 +4,8 @@
58 the source package, to keep things a bit more organised.
59 * Run tests at build time, now that python-txfixtures is in
60 ppa:launchpad/ubuntu/ppa (and >= zesty).
61+ * Drop qemu emulation support. It was quite unreliable, and we've had
62+ real hardware for a while.
63
64 -- Colin Watson <cjwatson@ubuntu.com> Tue, 25 Jul 2017 23:07:58 +0100
65
66
67=== modified file 'debian/control'
68--- debian/control 2017-07-28 10:12:15 +0000
69+++ debian/control 2017-07-28 11:12:54 +0000
70@@ -11,7 +11,6 @@
71 Depends: python-lpbuildd (=${source:Version}), python, debootstrap, dpkg-dev,
72 file, bzip2, sudo, ntpdate, adduser, apt-transport-https, lsb-release,
73 pristine-tar, python-apt, sbuild, ${misc:Depends}
74-Recommends: qemu-user-static
75 Description: Launchpad buildd slave
76 This is the launchpad buildd slave package. It contains everything needed to
77 get a launchpad buildd going apart from the database manipulation required to

Subscribers

People subscribed via source and target branches

to all changes: