ManimCommunity / ManimPango

Binding for Pango, to use with Manim.
https://manimpango.manim.community
MIT License
41 stars 13 forks source link

Symbol not found: ___darwin_check_fd_set_overflow #73

Closed mforbes closed 2 years ago

mforbes commented 2 years ago

With Mac OS X 10.14.6, I get the following error with "ManimPango v0.4.0.post1, Python v3.9.7":

Original error: dlopen(/.../lib/python3.9/site-packages/manimpango/cmanimpango.cpython-39-darwin.so, 2): Symbol not found: ___darwin_check_fd_set_overflow
  Referenced from: /.../lib/python3.9/site-packages/manimpango/.dylibs/libX11.6.dylib (which was built for Mac OS X 11.0)
  Expected in: /usr/lib/libSystem.B.dylib
 in /.../lib/python3.9/site-packages/manimpango/.dylibs/libX11.6.dylib

Most likely the key is that libX11.6.dylib (which was built for Mac OS X 11.0), and downgrading to v0.4.0.post0 fixes the issue.

Not sure if there is something to be done here, but is there any strategy for pinning appropriate versions in a platform dependent way, either with Conda, poetry, or pip?

naveen521kk commented 2 years ago

So something is wrong with the v0.4.0.post1 version and not v0.4.0.post0? I'll check the build logs to see if there is any issue.

naveen521kk commented 2 years ago

Hmm, yeah checking build log something is broken

  /usr/local/Cellar/libx11/1.7.3.1/lib/libX11-xcb.1.dylib
  /usr/local/Cellar/libx11/1.7.3.1/lib/libX11.6.dylib
  /usr/local/Cellar/libxau/1.0.9/lib/libXau.6.dylib
  /usr/local/Cellar/libxcb/1.14_1/lib/libxcb-render.0.0.0.dylib
  /usr/local/Cellar/libxcb/1.14_1/lib/libxcb-shm.0.0.0.dylib
  /usr/local/Cellar/libxcb/1.14_1/lib/libxcb.1.1.0.dylib
  /usr/local/Cellar/libxdmcp/1.1.3/lib/libXdmcp.6.dylib

These libs weren't(and shouldn't be) included in previous release. Thanks for reporting. I'll fix soon.

naveen521kk commented 2 years ago

This was because of a new version of meson, which added support for finding Xlib, so I have pinned the previous version for now.

naveen521kk commented 2 years ago

@mforbes can you test the wheels from here from CI run of #74?

mforbes commented 2 years ago

@naveen521kk Works with python 3.9 (I can generate some animations - did not run a full test suite though). It basically works with 3.7, 3.8, and 3.10 (I can import manim which typically raises the error, but my dependences make it hard to test full functionality quickly.)

Let me know if you want me to try to run a full test suite. I have not done that before, but probably could.

Thanks!

P.S. (Will this be released as post2 eventually?)

naveen521kk commented 2 years ago

Let me know if you want me to try to run a full test suite. I have not done that before, but probably could.

I don't think it's needed, just running import should be enough.

Will this be released as post2 eventually?

Yup.

naveen521kk commented 2 years ago

The new release(.post2) fixes this, right?

mforbes commented 2 years ago

Works for me. Thanks!