Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Symbol not found: ___darwin_check_fd_set_overflow #73

Closed
mforbes opened this issue Jan 27, 2022 · 8 comments · Fixed by #74
Closed

Symbol not found: ___darwin_check_fd_set_overflow #73

mforbes opened this issue Jan 27, 2022 · 8 comments · Fixed by #74

Comments

@mforbes
Copy link

mforbes commented Jan 27, 2022

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
Copy link
Member

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
Copy link
Member

naveen521kk commented Jan 27, 2022

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 added a commit that referenced this issue Jan 27, 2022
naveen521kk added a commit that referenced this issue Jan 27, 2022
It should detect issues like #73
@naveen521kk
Copy link
Member

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 added a commit that referenced this issue Jan 27, 2022
It should detect issues like #73
naveen521kk added a commit that referenced this issue Jan 27, 2022
naveen521kk added a commit that referenced this issue Jan 27, 2022
It should detect issues like #73
@naveen521kk
Copy link
Member

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

@mforbes
Copy link
Author

mforbes commented Jan 27, 2022

@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
Copy link
Member

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 added a commit that referenced this issue Jan 28, 2022
* wheel-build: Disable xft backend by default

It's useless for us. 
Fixes #73

* CI: use a old version of mac to test

It should detect issues like #73

* wheel-build: pin meson version

* wheels-build: disable xlib backend on cairo also
@naveen521kk
Copy link
Member

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

@mforbes
Copy link
Author

mforbes commented Jan 31, 2022

Works for me. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants