Merge lp:~cjwatson/launchpad/lpbuildd-egg into lp:launchpad

Proposed by Colin Watson
Status: Merged
Merged at revision: 17846
Proposed branch: lp:~cjwatson/launchpad/lpbuildd-egg
Merge into: lp:launchpad
Diff against target: 52 lines (+3/-4)
4 files modified
buildout.cfg (+0/-1)
lib/lp/buildmaster/tests/test_interactor.py (+1/-3)
setup.py (+1/-0)
versions.cfg (+1/-0)
To merge this branch: bzr merge lp:~cjwatson/launchpad/lpbuildd-egg
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+276663@code.launchpad.net

Commit message

Install launchpad-buildd as a Python dependency rather than expecting it as a system dependency.

Description of the change

Install launchpad-buildd as a Python dependency rather than expecting it as a system dependency. This goes with https://code.launchpad.net/~cjwatson/launchpad-buildd/build-sdist/+merge/276654, and has the same reasoning: it frees us from having to ask sysadmins to upgrade buildbots manually rather than being able to control the versions in use for ourselves, and it lets us get rid of some special buildout configuration to add /usr/lib/launchpad-buildd to sys.path.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'buildout.cfg'
2--- buildout.cfg 2015-08-21 13:09:09 +0000
3+++ buildout.cfg 2015-11-04 14:43:37 +0000
4@@ -53,7 +53,6 @@
5 # Make mailman built and installed in a more normal way.
6 extra-paths =
7 ${buildout:directory}/lib/mailman
8- /usr/lib/launchpad-buildd
9 include-site-packages = true
10 allowed-eggs-from-site-packages =
11 interpreter = py
12
13=== modified file 'lib/lp/buildmaster/tests/test_interactor.py'
14--- lib/lp/buildmaster/tests/test_interactor.py 2015-09-07 11:03:39 +0000
15+++ lib/lp/buildmaster/tests/test_interactor.py 2015-11-04 14:43:37 +0000
16@@ -509,11 +509,9 @@
17 self.assertEqual(['1.0', 'i386'], info[:2])
18 self.assertThat(
19 info[2],
20- # XXX cjwatson 2015-09-07: This should also include 'snap' once
21- # the buildbot slaves are upgraded to python-lpbuildd >= 133.
22 ContainsAll(
23 ('sourcepackagerecipe', 'translation-templates',
24- 'binarypackage', 'livefs')))
25+ 'binarypackage', 'livefs', 'snap')))
26
27 @defer.inlineCallbacks
28 def test_initial_status(self):
29
30=== modified file 'setup.py'
31--- setup.py 2015-06-17 04:53:19 +0000
32+++ setup.py 2015-11-04 14:43:37 +0000
33@@ -47,6 +47,7 @@
34 'python-keystoneclient',
35 'python-subunit',
36 'python-swiftclient',
37+ 'launchpad-buildd',
38 'launchpadlib',
39 'lazr.batchnavigator',
40 'lazr.config',
41
42=== modified file 'versions.cfg'
43--- versions.cfg 2015-08-03 07:06:45 +0000
44+++ versions.cfg 2015-11-04 14:43:37 +0000
45@@ -46,6 +46,7 @@
46 jsautobuild = 0.2
47 keyring = 0.6.2
48 kombu = 3.0.26
49+launchpad-buildd = 136
50 launchpadlib = 1.10.2
51 lazr.authentication = 0.1.1
52 lazr.batchnavigator = 1.2.11