Merge lp:~wgrant/meta-lp-deps/lp-soyuz-deps into lp:meta-lp-deps

Proposed by William Grant
Status: Merged
Approved by: Julian Edwards
Approved revision: not available
Merged at revision: not available
Proposed branch: lp:~wgrant/meta-lp-deps/lp-soyuz-deps
Merge into: lp:meta-lp-deps
Diff against target: 51 lines (+19/-4)
2 files modified
debian/changelog (+8/-0)
debian/control (+11/-4)
To merge this branch: bzr merge lp:~wgrant/meta-lp-deps/lp-soyuz-deps
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Michael Nelson (community) Approve
Gavin Panella (community) Approve
Review via email: mp+15836@code.launchpad.net
To post a comment you must log in.
Revision history for this message
William Grant (wgrant) wrote :

One of my pending LP branches needs a version of dpkg that supports the v3 formats. Since Hardy doesn't have a sufficiently new version, we need to keep a backport on all Soyuz machines.

I've created a new launchpad-soyuz-dependencies metapackage which depends on a new-enough dpkg. I've also moved the existing germinate and devscripts dependencies from launchpad-dependencies to launchpad-soyuz-dependencies, as again only the Soyuz machines need them.

launchpad-developer-dependencies also now depends on launchpad-soyuz-dependencies, as the test suite needs it.

Revision history for this message
William Grant (wgrant) wrote :

Just to be safe, I ec2tested without launchpad-soyuz-dependencies. The (quite reasonable -- xx-resetpassword-of-sso-account.txt is known to fail for non-Canonicalites) test results were as follows:

Tests with errors:
   lp.archiveuploader.tests.test_ppauploadprocessor.TestPPAUploadProcessor.testPPAUploadResultingInNoBuilds
   lp.archiveuploader.tests.test_uploadprocessor.TestUploadProcessor.testUploadResultingInNoBuilds
   testLimitedRun (lp.soyuz.scripts.tests.test_processpendingpackagediffs.TestProcessPendingPackageDiffsScript)
   testSimpleRun (lp.soyuz.scripts.tests.test_processpendingpackagediffs.TestProcessPendingPackageDiffsScript)
   testSimpleScriptRun (lp.soyuz.scripts.tests.test_processpendingpackagediffs.TestProcessPendingPackageDiffsScript)

Tests with failures:
   lib/lp/soyuz/tests/../doc/fakepackager.txt
   lib/lp/soyuz/tests/../doc/package-diff.txt
   lib/lp/services/openid/tests/../stories/xx-resetpassword-of-sso-account.txt

Revision history for this message
Gavin Panella (allenap) wrote :

This seems reasonable and good, and thank you for doing it! However, please can you ask a member of the Soyuz team to give it a once-over to be sure. I know almost nothing about packaging so I don't feel comfortable giving a blanket yes.

review: Approve
Revision history for this message
Michael Nelson (michael.nelson) wrote :

I'm sure bigjools will still take a look, but it's a learning opportunity for me too :)

My only thought was about devscripts - I had thought that code might depend on it for some build-from-branch stuff (but I guess that'll all be bzr plugins).

I'm assuming you've already tested your other LP branch with this dependency?

Thanks for getting that done William (and doing so carefully).

review: Approve
Revision history for this message
Julian Edwards (julian-edwards) :
review: Approve

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 2009-12-07 13:24:58 +0000
3+++ debian/changelog 2009-12-08 21:15:23 +0000
4@@ -1,3 +1,11 @@
5+launchpad-dependencies (0.60) karmic; urgency=low
6+
7+ * Split the Soyuz-only dependencies into launchpad-soyuz-dependencies.
8+ * launchpad-developer-dependencies depends on launchpad-soyuz-dependencies.
9+ * launchpad-soyuz-dependencies depends on dpkg >= 1.15.4, for v3 support.
10+
11+ -- William Grant <wgrant@ubuntu.com> Wed, 09 Dec 2009 07:45:37 +1100
12+
13 launchpad-dependencies (0.59) karmic; urgency=low
14
15 * Depend on xvfb, firefox, and xfonts-base in
16
17=== modified file 'debian/control'
18--- debian/control 2009-12-07 12:47:52 +0000
19+++ debian/control 2009-12-08 21:15:23 +0000
20@@ -13,10 +13,9 @@
21 python2.5-subversion, python-sqlite, ubuntu-keyring,
22 python2.5-imaging, bison, python-paramiko, python-mechanize,
23 python-openssl, graphviz, python-zopeinterface, python-celementtree,
24- python2.5-tickcount, germinate, devscripts, libxml2-utils,
25- python-epydoc, patchutils, openssh-blacklist,
26- openssh-client (>= 1:4.7p1-8ubuntu1.1), xsltproc, python-magic,
27- geoip-data | geoip-data-city-lite,
28+ python2.5-tickcount, libxml2-utils, python-epydoc, patchutils,
29+ openssh-blacklist, openssh-client (>= 1:4.7p1-8ubuntu1.1), xsltproc,
30+ python-magic, geoip-data | geoip-data-city-lite,
31 python-psycopg2 (>= 2.0.7+bzr361-0~CAT.8.04), python-pygments,
32 python-support, python-openid, python2.5-pysqlite2, zlib1g-dev,
33 ca-certificates, python-pyrex, libapr1-dev, libsvn-dev
34@@ -26,9 +25,17 @@
35 This package depends on all of the packages necessary to run the Launchpad
36 application.
37
38+Package: launchpad-soyuz-dependencies
39+Architecture: all
40+Depends: launchpad-dependencies (= ${source:Version}), dpkg (>= 1.15.4),
41+ germinate, devscripts
42+Description: Metapackage for Launchpad Soyuz packages
43+ This package depends on all of the packages necessary to run Soyuz.
44+
45 Package: launchpad-developer-dependencies
46 Architecture: all
47 Depends: launchpad-dependencies (= ${source:Version}), launchpad-database-dependencies (= ${source:Version}),
48+ launchpad-soyuz-dependencies (= ${source:Version}),
49 exuberant-ctags, bzr-pqm, python-pullparser, python-clientcookie,
50 postgresql-doc-8.3, postgresql-autodoc, tidy,
51 python-codespeak-lib, pylint, ssh, mhonarc,

Subscribers

People subscribed via source and target branches