cython/cython

Cythonize/Cython compilation fails to import/recognize `.pxd` from external modules that only have `__init__.so`

Open

#2.886 geöffnet am 4. März 2019

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Python (8.663 Stars) (1.517 Forks)batch import
Build Systemenhancementgood first issue

Beschreibung

When trying to use a .pxd from an external module and making sure it can be found on sys.path, Cythonize/Cython compilation will still fail to pick it up if that module only has __init__.cpython-36m-x86_64-linux-gnu.so but no __init__.py (which it obviously couldn't have, given the __init__ part is already that C extension).

As soon as I do touch __init__.py in that folder it's all picked up fine, but of course that's not really a meaningful thing to do and renders the module broken, which I think I shouldn't have to do just so I can import .pxd files from it, should I?

So this looks a bit like a Cython bug to me, tested with Cython 0.29.6

Contributor Guide

Tech Stack
pythonc
Domain
backendbuild system
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understanding of Cython's build processBasic knowledge of Python packaging
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
15
Research-Richtung
Investigate how Cython's import of .pxd files handles modules with only init .so (no init .py). Look at the import logic in Cython's compiler and Cythonize to understand why it fails. Check the condition that checks for init .py and see if it can be extended to also accept init .so as a valid marker for a package. Relevant files might include `Cython/Build/Dependencies.py` or `Cython/Compiler/Scanning.py`.