Skip to content
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

Fixes for OSX #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fixes for OSX #10

wants to merge 3 commits into from

Conversation

beadon
Copy link

@beadon beadon commented Mar 12, 2024

Not a complete fix, but an obvious fix for the missing OSX dependency.

@beadon
Copy link
Author

beadon commented Mar 12, 2024

Added a fix to find the eye-fi device on a MacOS system. Not special sauce, borrowed the logic from the Linux version and hacked away.

note the md5 change in this commit too , this appeared to need to be dereferenced correctly.

@beadon beadon changed the title added fcntl dependency for OSX Fixes for OSX Mar 12, 2024
@bjanssen
Copy link

bjanssen commented Nov 17, 2024

@beadon , did you get this to work (and on what machine)?
I tried with an M3 on Sonoma and an x86 on Sequoia and both get stuck in the read_from function (goto retry loop) giving
fd_flush(3) was unsuccessful(0), retrying (retry count)...
perror tells me Bad file descriptor.

Your code does compile on my macs, in contrast to the original master and the fork/PR by @btb .

Update: found an old version of @btb 's fork on my HD (with some minor edits). It was probably compiled on snow leopard. The binary does work on my Sequoia x86 machine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants