Skip to content

macOS: Check for macOS 10.10 using kCFCoreFoundationVersionNumber instead of AppKit#611

Merged
Youw merged 1 commit intolibusb:masterfrom
mrpippy:no_appkit
Aug 27, 2023
Merged

macOS: Check for macOS 10.10 using kCFCoreFoundationVersionNumber instead of AppKit#611
Youw merged 1 commit intolibusb:masterfrom
mrpippy:no_appkit

Conversation

@mrpippy
Copy link
Copy Markdown
Contributor

@mrpippy mrpippy commented Aug 15, 2023

Fixes: #587

@Youw Youw added the macOS Related to macOS backend label Aug 15, 2023
@Youw
Copy link
Copy Markdown
Member

Youw commented Aug 15, 2023

This looks perfect.
@nguyen-phillip can you confirm this fixes the issue in your environment?

@nguyen-phillip
Copy link
Copy Markdown

Yes, this should work -- the issue only manifests when linked against AppKit. Thanks!

@Youw
Copy link
Copy Markdown
Member

Youw commented Aug 15, 2023

should work

Would you be able to actually run and check?
Thanks.

@nguyen-phillip
Copy link
Copy Markdown

We've patched out this version check entirely as we don't need to support macOS predating 10.10. However the minimal repro code for the (still open) Apple bug only has problems when AppKit is linked in. CoreFoundation and IOKit are fine.

Copy link
Copy Markdown
Member

@Youw Youw left a comment

Choose a reason for hiding this comment

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

I wasn't able to reproduce the original #587 with current master. Maybe because I'm using M2 mac machine. Thus I cannot confirm if it really fixes the original issue or not.

@mrpippy can you confirm that you had the original issue reproduced locally and not reproducible after this change?

In any case - this make one dependency less compared to "before" which is a great thing on its own.

@Youw Youw merged commit 4009466 into libusb:master Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

macOS Related to macOS backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mac: hid_enumerate crash on macOS 14 when linked to AppKit and inside a fork

3 participants