Date: Wed, 28 Nov 2018 05:52:05 +0000 From: bugzilla-noreply@freebsd.org To: python@FreeBSD.org Subject: [Bug 233561] converters/py-rencode: Fails to configure if cython is installed Message-ID: <bug-233561-21822-27me86S8Q2@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-233561-21822@https.bugs.freebsd.org/bugzilla/> References: <bug-233561-21822@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D233561 --- Comment #2 from commit-hook@freebsd.org --- A commit references this bug: Author: koobs Date: Wed Nov 28 05:51:50 UTC 2018 New revision: 486079 URL: https://svnweb.freebsd.org/changeset/ports/486079 Log: converters/py-rencode: Fails to configure if cython is installed ports r482774 removed USE_PYTHON=3Dcython, likely due to it (cython) not being explicitly declared as a dependency in setup.py:*_requires. However, setup.py conditionally builds with cython if it is installed, but the 1.0.6 source distribution (sdist) does not contain a .pyx file to bui= ld with. This leads to a configure/build error when cython is installed: ValueError: 'rencode/rencode.pyx' doesn't match any files Upstream commit 5c928f14567fabc9efb8bbb8ac5e0eef03c61541 [1] via issue #25 [2] adds the required .pyx file to the sdist, which technically addresses the "build with cython from the sdist" issue, but does not fundamentally resolve the higher-level question: why build with cython wh= en a C source pre-processed by cython has already been produced for, and is contained in, the source distribution. A cython build (and the dependency on cython) does not appear to be necessary, nor intended [3][4], nor recommended [5], even after #25 is released, for sdist consumers. The conditional code to build with cython should not be included in setup.py, at least not as a default case, or without an explicit request by the user to use cython, as an optional build method, for sdist consume= rs (most downstream packagers and setuptools/pip users [6]). This change removes the conditional check for Cython, leaving the standard setuptools build_ext (with the packaged .c) file, as the build method. While I'm here: - Canonicalise COMMENT (match seutp.py:description) - Add test target with post-patch target to make the tests dir a module usable by a setup.py test (via test_suite directive) target. - Honour CFLAGS: Remove forced (appended) -O3 arg from setup.py [1] https://github.com/aresch/rencode/commit/5c928f14567fabc9efb8bbb8ac5e0eef03= c61541 [2] https://github.com/aresch/rencode/pull/25 [3] setup.py: "Error: sdist requires cython module to generate `.c` file." [4] dev-requirements.txt:Cython [5] https://cython.readthedocs.io/en/latest/src/userguide/source_files_and_comp= ilation.html#distributing-cython-modules [6] https://github.com/aresch/rencode/issues/11 PR: 233561 Reported by: John Hein <z7dr6ut7gs snkmail com> Approved by: koobs (python) Changes: head/converters/py-rencode/Makefile head/converters/py-rencode/files/ head/converters/py-rencode/files/patch-setup.py --=20 You are receiving this mail because: You are on the CC list for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-233561-21822-27me86S8Q2>