YoSTEALTH / Liburing

Liburing is Python + Cython wrapper around C Liburing, which is a helper to setup and tear-down io_uring instances.
https://pypi.org/project/liburing/
Creative Commons Zero v1.0 Universal
98 stars 3 forks source link

expected bytes, __common_define__ found #20

Closed ZisIsNotZis closed 2 months ago

ZisIsNotZis commented 2 months ago

Hi, I got a wired issue when running example:

Traceback (most recent call last):
  File "/home/t/ring.py", line 73, in <module>
    main()
  File "/home/t/ring.py", line 57, in main
    fd = open(ring, cqes, '/tmp/liburing-test-file.txt', O_CREAT | O_RDWR)
  File "/home/t/ring.py", line 15, in open
    io_uring_prep_openat(sqe, dir_fd, _path, flags, mode)
  File "src/liburing/file.pyx", line 146, in liburing.file.io_uring_prep_openat
TypeError: expected bytes, __common_define__ found

Any body know how to solve this? Thanks!

YoSTEALTH commented 2 months ago

deleted those messages, looks like spam?

Anyways @ZisIsNotZis let me see whats going on.

YoSTEALTH commented 2 months ago

@ZisIsNotZis fixed the bug https://github.com/YoSTEALTH/Liburing/commit/95769dc972a1dadd05aa8846012f153b33986dad

ZisIsNotZis commented 2 months ago

@ZisIsNotZis fixed the bug 95769dc

Thanks! Now it works! 🎉

I never thought you could respond this quickly LOL