-
Notifications
You must be signed in to change notification settings - Fork 333
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
v. 1.18.0: build error: 'kFSEventStreamCreateFlagFileEvents' was not declared in this scope #335
Comments
I surely will |
Hey, I don't have a system |
I am away from my PowerMac until tomorrow, but the build fails with the same error on buildbots: https://build.macports.org/builders/ports-10.6_x86_64-builder/builds/226218/steps/install-port/logs/stdio |
If I remember correctly, that entire class shouldn’t be compiled if that
API is not available. Or the check is buggy. I’ll look into it ASAP
…On Sat, 25 Jan 2025 at 20:49, Sergey Fedorov ***@***.***> wrote:
I am away from my PowerMac until tomorrow, but the build fails with the
same error on buildbots:
https://build.macports.org/builders/ports-10.6_x86_64-builder/builds/226218/steps/install-port/logs/stdio
I don’t see how configure can handle this, since the function is used
unconditionally (unlike a few other in the same source file).
—
Reply to this email directly, view it on GitHub
<#335 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABSWO2WHEGXH2II2U6RUOU32MPTE7AVCNFSM6AAAAABV3KAAQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMJUGA3TSOJQGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@emcrisostomo fsevents are supported back to 10.5, I think, and the code here seems to assume that at least partly it was building (and working?) on Leopard (otherwise why macros for it). |
In reality this is not a regression: even though macros that detect macOS version back to when FSEvents were introduced in |
@barracuda156 , I've pushed a branch with the required changes. I've tested both code paths on macOS-current, but I'm unable to test it on 10.5. Would you help me here? The branch is associated with this issue. I'm attaching a release tarball to make it easier for you to test it out if you're willing. |
@emcrisostomo Thank you! The build from e2e4a92 worked fine on 10.6 (where it failed before). I do not have a working set-up on earlier systems though. |
Thank you very much! 10.6 is good enough I guess. I'll merge it shortly. Could you try the basic functionality of the binary as well? |
Thank you. I will try how it works soon. |
@emcrisostomo Could you please take a look?
kFSEventStreamCreateFlagFileEvents
only exists in 10.7+:https://developer.apple.com/documentation/coreservices/1455376-fseventstreamcreateflags/kfseventstreamcreateflagfileevents
Also: https://trac.macports.org/ticket/71935
The text was updated successfully, but these errors were encountered: