[GitHub] asitstands commented on issue #10951: Fix broken cython build

[GitBox <gi...@apache.org>]

asitstands commented on issue #10951: Fix broken cython build
URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-391032043
 
 
   This PR does not work as expected. I made a mistake to set `MXNET_ENABLE_CYTHON` instead of `MXNET_ENFORCE_CYTHON` when I tested. An import error occurs if a python script using mxnet runs with `MXNET_ENFORCE_CYTHON=1`.
   ```
   Traceback (most recent call last):
     File "/pds/pds131/asitdepends/MXNet/usr/lib/python3.6/site-packages/mxnet-1.2.0-py3.6-linux-x86_64.egg/mxnet/ndarray/_internal.py", line 30, in <module>
       from .._cy3.ndarray import NDArrayBase, CachedOp
   ImportError: /pds/pds131/asitdepends/MXNet/usr/lib/python3.6/site-packages/mxnet-1.2.0-py3.6-linux-x86_64.egg/mxnet/_cy3/ndarray.cpython-36m-x86_64-linux-gnu.so: undefined symbol: NNGetLastError
   ```
   The .so files from the .pyx files are created but the linker does not find `libmxnet.so`. <strike>I guess that [this line](https://github.com/asitstands/incubator-mxnet/blob/4b347d818f71797528f62a65114b751488c59a99/python/setup.py#L72) is the source of the problem. However the cython compilation itself fails without that line. The compile error is
   ```
   error: each element of 'ext_modules' option must be an Extension instance or 2-tuple
   ```
   as reported in #10295.</strike>

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services