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

Subscribers

People subscribed via source and target branches