lxml requirement is unused

Bug #1252344 reported by Alex Meade
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Alex Meade

Bug Description

requirements.txt currently has lxml listed as a requirement. Glance does not make use of lxml (or xml in general) so this should be removed. It is also a burden when installing requirements on a system with limited memory since compiling lxml takes a significant amount.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

Fix proposed to branch: master
Review: https://review.openstack.org/56991

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/56991
Committed: http://github.com/openstack/glance/commit/b7f9120cd4623d8284c4edfea760ef10e1193492
Submitter: Jenkins
Branch: master

commit b7f9120cd4623d8284c4edfea760ef10e1193492
Author: Alex Meade <email address hidden>
Date: Mon Nov 18 16:54:52 2013 +0000

    Remove lxml requirement

    Fixes bug 1252344

    Change-Id: Iaa10b7c78bf03b00c84a27592265bb79203df376

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.