[CASSANDRA-11630] Make cython optional in pylib/setup.py

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 2.1.15, 2.2.7, 3.0.6, 3.6
• Component/s: Legacy/Tools
• Severity: Normal
• Assignee: Stefania Alborghetti
• Reporter: Stefania Alborghetti

Authors

Stefania Alborghetti

Reviewers

T Jake Luciani

• Tester: Michael Shuler
• Created: 22/Apr/16 01:27
• Updated: 16/Apr/19 09:30
• Resolved: 27/Apr/16 00:50

Description

When building deb packages, we currently run this line:

cd pylib && python setup.py install --no-compile --install-layout deb \
		--root $(CURDIR)/debian/cassandra

Since CASSANDRA-11053 was introduced, this will build the cython extensions for copyutil.py.

We should change setup.py so that when we specify --no-compile then the cython extensions are not built, in a similar way to what is done for the Python driver.

Issue Links

• is depended upon by: CASSANDRA-11573 cqlsh fails with undefined symbol: PyUnicodeUCS2_DecodeUTF8
• is depended upon by: CASSANDRA-10853 deb package migration to dh_python2
• is related to: CASSANDRA-11053 COPY FROM on large datasets: fix progress report and optimize performance part 4
• is related to: CASSANDRA-11574 clqsh: COPY FROM throws TypeError with Cython extensions enabled
• links to: 2.1 patch