-
Notifications
You must be signed in to change notification settings - Fork 588
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
error: implicit declaration of function 'fcntl' [-Werror,-Wimplicit-function-declaration] #782
Comments
@ryandesign You are right about including But then it bumps into non-existing
|
fixed in 044e6bd |
It looks like this change (or possibly some element of the fix for #783) has broken builds on newer macOS 10.13 and macOS 10.14, as there is now a compile error on
I'm assuming the patch file MacPorts is carrying from c-ares 1.30 to main branch is a correct reflection of the commits to land fixes for #782/#783. |
@sambthompson The patch is here: https://github.com/macports/macports-ports/blob/716024da0953ba766d759383640212bb0fa63665/net/c-ares/files/patch-fix-build-pre-10.12.diff I believe, I only removed changes to README, because that file did not exist, so it failed to apply. |
Thanks for clarifying; I had only noted the addition of |
Maybe it expects to see |
Isn't that usually a combination of compiler version and language version? I'm not across C-99 and later versions enough to comment on changes to macro syntax or how boolean evaluation works for |
The build on 10.13 and 10.14 was already broken in 1.30.0. Please file a separate issue; it is unrelated to this one. |
Sorry, my recollection was that all builds 10.12 and newer were working (from looking at the port health) for 1.30.0, before the fixes for #782 and #783 were downstreamed via the patch. This is why the original issue 70207 was filed on MacPorts against 10.11 and older (as well as noting separate issue with 10.7 and earlier that you filed this issue here for). |
From the builds page you can see that builds on 10.13 and 10.14 failed on June 16 after the fixes for older systems went in, but also on June 12 when the update to 1.30.0 was first committed. |
Wish I had a screen-shot of the port-health screen to check my recollection. Anyway, have now reported the UNISTD_H issue for 10.13 and 10.14 as #787. |
please try 5804b30 |
c-ares 1.31.0 is now out which contains the legacy mac fixes |
Thanks so much for this, Brad; have opened update request at MacPorts to update to this new release. |
1.30.0 fails to build on Mac OS X 10.7:
And on Mac OS X 10.6:
fcntl
is defined in <fcntl.h> but you don't appear to have#include
d that header in ares_event_configchg.c.The text was updated successfully, but these errors were encountered: