#197: Add FeedParser as external requirement instead of shipping with it

Contents

1. Motivation
2. Proposal
3. Implementation
4. Deliverables
5. Risks
6. Participants

Proposed by

Christian Scholz

Seconded by

Jens Klein

Proposal type

Architecture

State

being-discussed

Motivation

It's simply not elegant to ship with an external library inside the Plone source code. Now that we have an egg story it is easy to make the plone.app.portlets egg require this library and install it automatically.

Proposal

Change setup.py of plone.app.portlets to have feedparser in the install_requires section. Then it should be removed from the source.

Implementation

see proposal

Deliverables

new setup.py for plone.app.portlets and removal of feedparser.py

It's open though if we should put a deprecation warning in there for a limited time (see http://pypi.python.org/pypi/zope.deprecation/3.4.0).

Risks

- somebody might import it directly from this location (which is very unlikely though as you need to know that it's there)
- this will break upgrades on sites which do not have feedparser installed and who are doing a manual install

Participants

Christian Scholz