Merge lp:~allenap/maas/lxml-not-a-test-dep into lp:maas/trunk
Proposed by
Gavin Panella
on 2012-09-24
| Status: | Merged |
|---|---|
| Approved by: | Gavin Panella on 2012-09-24 |
| Approved revision: | 1058 |
| Merged at revision: | 1058 |
| Proposed branch: | lp:~allenap/maas/lxml-not-a-test-dep |
| Merge into: | lp:maas/trunk |
| Diff against target: |
23 lines (+0/-2) 2 files modified
buildout.cfg (+0/-1) versions.cfg (+0/-1) |
| To merge this branch: | bzr merge lp:~allenap/maas/lxml-not-a-test-dep |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Raphaël Badin (community) | 2012-09-24 | Approve on 2012-09-24 | |
|
Review via email:
|
|||
Commit Message
Remove lxml from buildout's control.
lxml is a production dependency, and as such it should no longer be managed by buildout.
To post a comment you must log in.
review:
Approve
| John A Meinel (jameinel) wrote : | # |
| Julian Edwards (julian-edwards) wrote : | # |
Don't forget these simple things for the precise branch too.


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 9/24/2012 2:48 PM, Gavin Panella wrote: /code.launchpad .net/~allenap/ maas/lxml- not-a-test- dep/+merge/ 125969
> 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:/
>
>
review: approve
We'd like it for the lshw code as well.
John
=:->
-----BEGIN PGP SIGNATURE----- www.enigmail. net/
gRi0ACgkQJdeBCY SNAAP1VgCcCi30U Cvn9jo4EK6CSlXM w+AK aUu+hZ8F/ cKEbUVdu
Version: GnuPG v1.4.12 (Cygwin)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAlB
PkYAniblUII8bsW
=7g4S
-----END PGP SIGNATURE-----