Skip to content

Fix for Apple Silicon#7

Merged
minrk merged 3 commits intominrk:masterfrom
geohot:master
Dec 1, 2020
Merged

Fix for Apple Silicon#7
minrk merged 3 commits intominrk:masterfrom
geohot:master

Conversation

@geohot
Copy link
Copy Markdown
Contributor

@geohot geohot commented Nov 28, 2020

Fixing the segfault in IPython

appnope/_nope.py Outdated

# Needed to init ObjC, otherwise C('NSString') returns None
# TODO: Investigate why and include a better fix.
from urllib.request import urlopen
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of this line, we need to do _ = ctypes.cdll.LoadLibrary(ctypes.util.find_library('Foundation')) after the objc library load. I think this should have been there all along (as the framework to interface with c code).

@geohot
Copy link
Copy Markdown
Contributor Author

geohot commented Nov 28, 2020

To whoever maintains this, I feel good about this PR now with the hack removed.

@minrk
Copy link
Copy Markdown
Owner

minrk commented Dec 1, 2020

Awesome, thanks! Will release later today.

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 this pull request may close these issues.

3 participants