TRASAL / psrdada-python

Python bindings to the PSRDada ringbuffer implementation
https://trasal.github.io/psrdada-python/index.html
Other
9 stars 8 forks source link

make error #7

Open robin-pl opened 5 months ago

robin-pl commented 5 months ago

.local/lib/python3.8/site-packages/Cython/Build/Dependencies.py", line 1321, in cythonize_one raise CompileError(None, pyx_file) Cython.Compiler.Errors.CompileError: psrdada/reader.pyx

robin-pl commented 5 months ago

we build the psrdada_python in the ubuntu22 ,python3.10,but met some problmes, resolve it by: 1,cd psrdada modified dada_hdu.pxd ctypedef unsigned long int uint64_t to cdef extern from "stdint.h": ctypedef unsigned long int uint64_t

ctypedef unsigned long key_t to cdef extern from "stdint.h": ctypedef unsigned long key_t 2, modified the setup line 1 to #!/usr/bin/env python3

3,cd psrdada python3 -m cython *.pyx cd .. make

jiskattema commented 5 months ago

Thanks for reporting the issue! I tried installing on my laptop (Fedora 39, python 3.12). I had to fix some errors with cython not finding dada_hdu.pxd, but i cannot reproduce your error. Can you try again with current master version, and copy-paste the error in more detail?

To respond to your points:

  1. The first change seems reasonable: 'unsigned long int' is defined in 'stdint.h' on my system. The 'key_t' supposedly comes from "sys/ipc.h" -- see 'man ftok', so i prefer to get it from there instead.
  2. seems reasonable, i've changed the #! env line in setup to python3.

I've created a new branch can you see if that works?