Merge lp:~cjwatson/meta-lp-deps/frontend-dependencies into lp:meta-lp-deps

Proposed by Colin Watson
Status: Merged
Merged at revision: 177
Proposed branch: lp:~cjwatson/meta-lp-deps/frontend-dependencies
Merge into: lp:meta-lp-deps
Diff against target: 49 lines (+19/-1)
2 files modified
debian/changelog (+9/-0)
debian/control (+10/-1)
To merge this branch: bzr merge lp:~cjwatson/meta-lp-deps/frontend-dependencies
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+331760@code.launchpad.net

Commit message

Add a new launchpad-frontend-dependencies metapackage, currently
equivalent to launchpad-dependencies plus nodejs; this should be
sufficient to run "make build", which builds Launchpad's JavaScript code
as well as its Python code.

Description of the change

I previously added nodejs to launchpad-developer-dependencies in what I thought at the time was obvious support of https://code.launchpad.net/~cjwatson/launchpad/yarn/+merge/327823, but immediately afterwards I realised that that's not quite right, because we need it on the frontend machines too so that they can run "make build". I don't want to add it to all Launchpad hosts when most of them don't need it. This seems like a reasonable compromise: we'll have to get IS to install the new launchpad-frontend-dependencies package on our frontends, but that's easy enough.

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 'debian/changelog'
2--- debian/changelog 2017-10-01 14:25:53 +0000
3+++ debian/changelog 2017-10-04 01:32:49 +0000
4@@ -1,3 +1,12 @@
5+launchpad-dependencies (0.132) UNRELEASED; urgency=medium
6+
7+ * Add a new launchpad-frontend-dependencies metapackage, currently
8+ equivalent to launchpad-dependencies plus nodejs; this should be
9+ sufficient to run "make build", which builds Launchpad's JavaScript code
10+ as well as its Python code.
11+
12+ -- Colin Watson <cjwatson@ubuntu.com> Wed, 04 Oct 2017 02:24:29 +0100
13+
14 launchpad-dependencies (0.131) xenial; urgency=medium
15
16 * Add gnupg to launchpad-dependencies, just in case it's dropped from
17
18=== modified file 'debian/control'
19--- debian/control 2017-10-01 14:05:00 +0000
20+++ debian/control 2017-10-04 01:32:49 +0000
21@@ -47,6 +47,7 @@
22 launchpad-database-dependencies (= ${source:Version}),
23 launchpad-messagequeue-dependencies (= ${source:Version}),
24 launchpad-soyuz-dependencies (= ${source:Version}),
25+ launchpad-frontend-dependencies (= ${source:Version}),
26 exuberant-ctags, bzr-pqm, geoip-data-city-lite,
27 pgbouncer (>= 1.6) | pgbouncer-with-disconnect, postgresql-autodoc, tidy,
28 python-codespeak-lib | python-py,
29@@ -55,7 +56,6 @@
30 xvfb, firefox, intltool, python-pocket-lint,
31 python-lpbuildd (>= 88),
32 python-html5-browser,
33- nodejs,
34 ${misc:Depends}
35 Recommends: pyflakes, subunit, python-boto, bzr-lpreview-body,
36 postgresql-doc-9.3
37@@ -118,3 +118,12 @@
38 Description: Metapackage for Launchpad messagequeue packages
39 This metapackage depends on all of the packages necessary to run a Launchpad
40 messagequeue server.
41+
42+Package: launchpad-frontend-dependencies
43+Architecture: all
44+Depends: launchpad-dependencies (= ${source:Version}),
45+ nodejs,
46+ ${misc:Depends}
47+Description: Metapackage for Launchpad frontend dependencies
48+ This metapackage depends on all of the packages necessary to run a Launchpad
49+ frontend server, including building Launchpad's JavaScript.

Subscribers

People subscribed via source and target branches