-
Notifications
You must be signed in to change notification settings - Fork 22
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
Build fails on macOS: error: unknown type name 'ssize_t'; error: 'SOL_SOCKET' undeclared (first use in this function); etc. #20
Comments
Some of these are fixable:
This fixes undefined
(I am not sure if
@eyedeekay @obscuratus Can this be addressed? |
Sorry about the delay, I will figure this out when I get back to my office where my Mac is. |
@eyedeekay Greatly appreciated, thank you. |
Not really sure why I was mentioned here :) |
@obscuratus Sorry, feel free to ignore. I just looked who have committed to the source, and tagged to get some attention to the issue, which was open for some months. |
any news foe a fix? |
I'll get it as soon as I can in the next few days, I'm not normally a Mac user so I'm a shade unfamiliar with the tooling. |
no fix yet? |
Hi, when comes the fix to master? |
As soon as I can get to it, probably sometime around the 19th, I'm pretty swamped with the upcoming release right now and need a little more time. |
Sounds good! I just back to my PowerPC hardware, and right into the disaster of broken OpenBLAS build and a need to update/fix quite a number of related ports. So will also be busy for coming 3–4 days. |
No surprise here, but just for the record: here is the failure on 10.6 with gcc13:
|
OK I'm almost through the release marathon and ready to get back to this tomorrow. Talk soon. |
@eyedeekay Awesome! |
So I've applied the fixes you suggested and now the last thing to do before I merge macos-build-fixes is to decide what to do with sysinfo. IMO our use of it is kind of superfluous, we don't need it for the thing we're using it for so I'm just going to take that part and do something cross-platform there instead. Sorry about all the delays, been a rough week. |
@eyedeekay Sorry to disturb, but any luck with fixing this? |
Yeah I just need to get back to my Mac workstation again to finish up. |
Great to hear, will be waiting for that. |
@barracuda156 Sorry for the delay my Mac is broken, the PR here: #23 should resolve it but I will need you to help test it. |
@eyedeekay Thank you! Will try that today and let you know. |
@eyedeekay Does not work yet: #23 (comment)
|
Thanks much, not having a working Mac to test on has made this tricky, I'll try to iterate more quickly to help us catch them all. |
Added your changes and finished up with some of my own which should finish, please take a look and test when you can thank you. |
Thank you for an update! With new patches:
( Re Other stuff needs related headers, perhaps something like here: https://gist.github.com/lemonmojo/1b7f957aed60a65c121e8067d5d93483 |
@eyedeekay By the way, maybe make a new release? Been quite a while. |
The text was updated successfully, but these errors were encountered: