[PROTON-1330] Include the C sources in the python source distribution

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: proton-0.15.0
• Fix Version/s: proton-0.16.0
• Component/s: python-binding
• Assignee: Ken Giusti
• Reporter: Ken Giusti
• Created: 20/Oct/16 13:04
• Updated: 13/Dec/16 18:53
• Resolved: 28/Nov/16 20:03

Description

setup.py in the python bindings will download the proton-c sources from the apache dist url if proton is not installed.

The source tarballs hosted at apache dist may be deleted at any time by apache infrastructure maintainers. This will lead to failures when installing python via setup.py (as is done via pip for the pypi packages).

We should modify the cmake build of the python bindings to package up the necessary proton-c source files for inclusion in the python bindings package. Instead of fetching the necessary sources from apache, setup.py can simply build and install the proton library using the bundled sources.

Issue Links

• is depended upon by: PROTON-1345 [python] Avoid hardcoding the list of C files to be compiled in setup.py
• links to: GitHub Pull Request #85
• links to: GitHub Pull Request #88