ext/_yaml.c is missing in 4.1 tarball

[dfroger]

Installing pyyaml 3.12 without cython do build the _yaml extension, but installing 4.1 do not:

python -m venv venv3.12
./venv3.12/bin/pip install pyyaml==3.12

python -m venv venv4.1
./venv4.1/bin/pip install pyyaml==4.1

find . -name '_yaml*'
./venv3.12/lib/python3.6/site-packages/_yaml.cpython-36m-x86_64-linux-gnu.so

The problem is that in https://pypi.org/simple/pyyaml/ PyYAML-3.12.tar.gz contains ext/_yaml.c, but PyYAML-4.1.tar.gz do not, so cython is required for 4.1 to build the extension. (.pyx -> .c)

(question also asked in #179 but the issue was about another problem and is closed)

Could it be possible to make a new 4.1 tarball including ext/_yaml.c?

[ingydotnet]

Will address soon. #179 (comment)

[asottile]

@ingydotnet the usual way to fix this is to ensure cython is installed when uploading the release sdist -- a simple version bump and re-upload would fix this issue (and an assertion in setup.py that cython is required during sdist would prevent it in the future)

[asottile]

for instance, here's what shapely does

[perlpunk]

@asottile I created #188, thanks
unfortunately, now travis-ci complains: https://travis-ci.org/yaml/pyyaml/builds/397951551 because of missing cython

[asottile]

@perlpunk I sent a PR to your branch: perlpunk#1