launchpad-reviewers team mailing list archive

[Gavin Panella <gavin.panella@xxxxxxxxxxxxx>]

Gavin Panella has proposed merging lp:~allenap/maas/lxml-not-a-test-dep into lp:maas.

Commit message:
Remove lxml from buildout's control.

lxml is a production dependency, and as such it should no longer be managed by buildout.

Requested reviews:
  MAAS Maintainers (maas-maintainers)

For more details, see:
https://code.launchpad.net/~allenap/maas/lxml-not-a-test-dep/+merge/125969
-- 
https://code.launchpad.net/~allenap/maas/lxml-not-a-test-dep/+merge/125969
Your team MAAS Maintainers is requested to review the proposed merge of lp:~allenap/maas/lxml-not-a-test-dep into lp:maas.

=== modified file 'buildout.cfg'
--- buildout.cfg	2012-09-22 15:38:25 +0000
+++ buildout.cfg	2012-09-24 10:46:13 +0000
@@ -33,7 +33,6 @@
 test-eggs =
   coverage
   fixtures
-  lxml
   mock
   nose
   nose-subunit

=== modified file 'versions.cfg'
--- versions.cfg	2012-09-22 15:38:25 +0000
+++ versions.cfg	2012-09-24 10:46:13 +0000
@@ -9,7 +9,6 @@
 Jinja2 = 2.6
 Pygments = 1.4
 Sphinx = 1.1.3
-lxml = 2.3.5
 
 [versions-dev]
 # Bug 251 is problematic in 0.9.2.