Build not working under windows about silx HOT 8 CLOSED

@PiRK Review required.

from silx.

Not sure if it is an issue or not.

• If Cython is installed on the machine, the generated code works.
• If Cython is not installed on the machine, the supplied code does not work.

from silx.

I'll have a look at the cython issue. I probably need to adjust the list of source files in io/setup.py if os.environ["WITH_CYTHON"] == "False" is set in the general setup.py, to remove the pyx file and include the .c file.

from silx.

File extensions don't seem to be the problem, as fake_cythonize (https://github.com/silx-kit/silx/blob/master/setup.py#L296) already takes care of replacing .pyx with .c in the absence of cython.

from silx.

On Sun, 06 Mar 2016 23:43:17 -0800
Pierre Knobel [email protected] wrote:

File extensions don't seem to be the problem, as fake_cythonize (https://github.com/silx-kit/silx/blob/master/setup.py#L296) already takes care of replacing .pyx with .c in the absence of cython.

Look at the version of cython used and enforce the minimum version.

Armando, which version have you been using ?

from silx.

• If I am not mistaken, I have been using cython > 0.17 in order to use memory views.
• FreeART wraps more advanced C++ features and I think it needs 0.22 minimum.
• I do not anticipate problems with the Cython version requirements if we make sure we distribute generated code.

from silx.

is this issue fixed ?

from silx.

We fixed the problem preventing building when cython isn't installed (#13). The previous issue related to not using os.path has also been also fixed (#10).

from silx.