-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Workaround copy bug for APFS volumes #2032
Conversation
Drop down to the C api for copying files so we can avoid using the APFS cloning functionality, which has a bug that can result in invalid binaries
If anyone knows how to programatically check whether we're on an APFS volume or not, I could make that check and only do this modified copy in that situation. |
Awesome work on this, great stuff!
Took some wrangling, but Give me your comments on those commits, and pending your approval (of course), I can just push them right into this PR… |
Left some comments on the commits. Thanks for looking into the volume check |
Good notes on |
…copyItem(at:to:avoiding·rdar·32984063:)`. Comments migrated to Quick Help markup.
Looks good, you just missed changing the call site: Then I think we're good to go |
I don't love that we have to do this, but it seems like this is a reasonable workaround. Thanks for tackling it! I'll let the two of you keep going with this and merge it. 👍 |
I think we've each reviewed each others portion so I'm gonna go ahead and merge this 🎉 |
Fixes #1982
Drop down to the C api for copying files so we can avoid using the APFS
cloning functionality, which has a bug that can result in invalid
binaries