-
Notifications
You must be signed in to change notification settings - Fork 0
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
upgrading c-api cmake #63
Conversation
I have the suspicious that |
tbh, I think the c interface might be simple enough to skip the complexity introduced by cmake... I haven't had the mac the last few days, I'll try it out and see whether I can fix it. |
The problem you are seeing is due to I'd add it as a comment to the readme. There's another problem in mac os when running under virtual environments, but it is also a "non-problem" in the sense that a long as the executable is in |
updated the readme, the problem with |
Co-authored-by: Stefano Carrazza <scarrazza@users.noreply.github.com>
Anyway, I think we can merge this given that also linux needs those changes in order to compile successfully. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me. I did test it in Linux and it works.
This fixes compilation and installation issues on linux and mac, however on mac I cannot run the example using python3 from Xcode:
@scarlehoff does this works for you with homebrew?