Merge lp:~cjwatson/launchpad-buildd/livefs-ltsp into lp:launchpad-buildd

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 119
Merged at revision: 118
Proposed branch: lp:~cjwatson/launchpad-buildd/livefs-ltsp
Merge into: lp:launchpad-buildd
Diff against target: 30 lines (+12/-0)
2 files modified
buildlivefs (+5/-0)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~cjwatson/launchpad-buildd/livefs-ltsp
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+224306@code.launchpad.net

Commit message

Install ltsp-server (but not its Recommends) for livefs builds, as Edubuntu needs it.

Description of the change

Edubuntu i386 builds have regressed with the new Launchpad-based builds because ltsp-build-client is unavailable in the chroot:

  https://launchpad.net/~ubuntu-cdimage/+livefs/ubuntu/utopic/edubuntu/+build/97

This was previously only handled by make-chroot.sh in canonical-is-puppet installing ltsp-server in i386 livefs chroots, which was obscure enough that I didn't notice it.

I considered inventing a way for livefs jobs to request additional build-dependencies. However, Stéphane pointed out that ltsp-server without its Recommends is tiny, so it doesn't seem worth it for this.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) wrote :

This will probably need to be revised later on, as ltsp-server is a fairly non-core package that probably won't be pulled into the rtm derived distro, for example. But it unbreaks things for now.

review: Approve (code)
119. By Colin Watson

Confine the ltsp-server ugliness to i386.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'buildlivefs'
2--- buildlivefs 2014-06-15 23:40:27 +0000
3+++ buildlivefs 2014-06-24 14:59:40 +0000
4@@ -109,6 +109,11 @@
5
6 def install(self):
7 self.chroot(["apt-get", "-y", "install", "livecd-rootfs"])
8+ if self.options.arch == "i386":
9+ self.chroot([
10+ "apt-get", "-y", "--no-install-recommends", "install",
11+ "ltsp-server",
12+ ])
13 if self.options.locale is not None:
14 self.chroot([
15 "apt-get", "-y", "--install-recommends", "install",
16
17=== modified file 'debian/changelog'
18--- debian/changelog 2014-06-22 15:12:20 +0000
19+++ debian/changelog 2014-06-24 14:59:40 +0000
20@@ -1,3 +1,10 @@
21+launchpad-buildd (124) UNRELEASED; urgency=medium
22+
23+ * Install ltsp-server (but not its Recommends) for i386 livefs builds, as
24+ Edubuntu needs it.
25+
26+ -- Colin Watson <cjwatson@ubuntu.com> Tue, 24 Jun 2014 14:58:16 +0100
27+
28 launchpad-buildd (123) hardy; urgency=medium
29
30 * Fix handling of livefs builds for the -proposed pocket.

Subscribers

People subscribed via source and target branches

to all changes: