Install failed with Cython 0.25 about pyyaml HOT 12 OPEN

Hit by the same issue, building on travis fails for osx:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/170573675/log.txt

.....
      File "/Users/travis/build/maartenbreddels/vaex-wheels/venv/lib/python2.7/site-packages/Cython/Distutils/build_ext.py", line 19, in finalize_options

        self.distribution.ext_modules)

      File "/Users/travis/build/maartenbreddels/vaex-wheels/venv/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 809, in cythonize

        aliases=aliases)

      File "/Users/travis/build/maartenbreddels/vaex-wheels/venv/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 752, in create_extension_list

        **kwds))

    TypeError: __init__() takes exactly 6 arguments (3 given)

Using cython==0.24.1 seems to work.

from pyyaml.

I can't find the parameters that are required documented in Cython (feature_name, feature_description, feature_check). I won't have the time tonight to research this, but I hope to get to it before the weekend.

from pyyaml.

Would be great 👍

from pyyaml.

These parameters are in yaml's custom Extension class. This was due to the switch to use cythonize(...) by default (which I'm planning on temporarily reverting).

from pyyaml.

@robertwb so, we're going to have problems when you switch to it anyway. Can you help me understand how I might go about fixing this? I'm not intimately familiar with cython or distutils. Thanks.

from pyyaml.

Looks like you have quite the complicated setup.py file. This is going to take some thought, but I'll keep pyyaml in mind while rewriting https://cython.readthedocs.io/en/latest/src/userguide/source_files_and_compilation.html#

from pyyaml.

FYI, if you get travis ci up and running, it'd be easier for me to play with.

from pyyaml.

@robertwb Yeah, I've been trying to get a hold of someone who can add that. In the meantime, I assume we should be using something like https://cython.readthedocs.io/en/latest/src/userguide/source_files_and_compilation.html#multiple-cython-files-in-a-package ?

from pyyaml.

Is there any quick fix for the above? I got the same error while installing Keras on a Ubuntu 14.04 32 bit machine. Any help would be appreciated.

from pyyaml.

@robertwb we have Travis CI set up now.

from pyyaml.

from pyyaml.

I see, is this an easy fix, @sigmavirus24 ?

from pyyaml.