Ticket #1796 (closed enhancement: fixed)

Opened 2 years ago

Last modified 23 months ago

Get rid of lxml dependency

Reported by: dread Owned by: ross
Priority: awaiting triage Milestone: ckan-future
Component: ckan Keywords:
Cc: rgrp, kindly Repository: ckan
Theme: none

Description

lxml is used in CKAN core in two places:

  • ckan/lib/helpers.py:239 Parsing Markdown and ensuring it is valid XML (i.e. tags close nicely so the HTML of the rest of the CKAN page is not disruptable).
  • reading the SOLR config XML to determine what version it is.

Both of these could be done with xml.dom built into Python and would save us a sizeable and slightly problematic dependency.

lxml is needed by these extensions: ckanext-inspire, ckanext-harvest, ckanext-csw & ckanext-wordpresser, so we'd have to add this to their requirements.

Change History

comment:1 Changed 23 months ago by ross

  • Owner set to ross
  • Status changed from new to accepted

LXML removed, update requirements on the extensions

comment:2 Changed 23 months ago by ross

  • Status changed from accepted to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.