kivy / cymunk

Cython port of Pymunk
http://readthedocs.org/docs/cymunk/en/latest/
MIT License
45 stars 29 forks source link

- change cymunk/cymunk to cymunk.cymunk in ext_modules #30

Closed maho closed 9 years ago

maho commented 9 years ago

I'm not 100% sure what does it mean, but seems that without this change, I have following error:

(kivent2)maho@dlaptop:~/workspace/khamster/pypackages2/cymunk$ python setup.py build_ext --inplace
running build_ext
cythoning cymunk/cymunk.pyx to cymunk/cymunk.c

Error compiling Cython file:
------------------------------------------------------------
...
include "constraint.pxi"
^
------------------------------------------------------------

cymunk/cymunk.pyx:1:0: 'cymunk/cymunk' is not a valid module name
building 'cymunk/cymunk' extension
x86_64-linux-gnu-gcc -pthread -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fno-strict-aliasing -D_FORTIFY_SOURCE=2 -g -fstack-protector-strong -Wformat -Werror=format-security -fPIC -Icymunk/Chipmunk-Physics/include -Icymunk/Chipmunk-Physics/include/chipmunk -I/usr/include/python2.7 -c cymunk/cymunk.c -o build/temp.linux-x86_64-2.7/cymunk/cymunk.o -std=c99 -ffast-math -fPIC -DCHIPMUNK_FFI
cymunk/cymunk.c:1:2: error: #error Do not use this file, it is the result of a failed Cython compilation.
 #error Do not use this file, it is the result of a failed Cython compilation.
  ^
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
Kovak commented 9 years ago

works for me and seems more sensible. Not certain why either cymunk/cymunk or cymunk.cymunk both works on my system.

maho commented 9 years ago

W dniu 13.05.2015 o 23:21, Kovak pisze:

works for me and seems more sensible. Not certain why either cymunk/cymunk or cymunk.cymunk both works on my system.

Probably beause you have already cythonized .pyx file.

Kovak commented 9 years ago

Oh great call that is precisely it. Thanks for the PR and catching the error!