-
-
Notifications
You must be signed in to change notification settings - Fork 169
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
Mac updates #384
Open
ahknight
wants to merge
15
commits into
trapexit:master
Choose a base branch
from
ahknight:mac-updates
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Mac updates #384
Commits on Apr 5, 2017
-
Configuration menu - View commit details
-
Copy full SHA for d9ae6df - Browse repository at this point
Copy the full SHA d9ae6dfView commit details -
As the position argument isn’t carried all the way down resource forks won’t work properly (though it easily could be, that would change the internal API for everyone).
Configuration menu - View commit details
-
Copy full SHA for 1aea1d4 - Browse repository at this point
Copy the full SHA 1aea1d4View commit details -
OSX doesn’t like AT_FDCWD in it’s fcntl, but getcwd does the job. Also fixed an issue with absolute paths in the process.
Configuration menu - View commit details
-
Copy full SHA for 39b7772 - Browse repository at this point
Copy the full SHA 39b7772View commit details -
Updating return/variable types
After running Clang’s analyzer on it there were quite a few places where there were mis-matched store sizes and signs. I updated them so that the underlying code more closely-matches the OS APIs and the type change happens at the FUSE connection. In this way, if FUSE ever updates the API to actually match the APIs of modern OSes, it’s a fix in one place. Also, I updated read/write to return the total read/write value the OS returned rather than blindly return the request count.
Configuration menu - View commit details
-
Copy full SHA for d1aec14 - Browse repository at this point
Copy the full SHA d1aec14View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6cfd4d5 - Browse repository at this point
Copy the full SHA 6cfd4d5View commit details -
Adding position arg to xattr APIs
Finder copies are broken without this argument, even if it’s fixed to 0. Upside: it reduces the number of times we platform-check in the xattr code.
Configuration menu - View commit details
-
Copy full SHA for 59ece5f - Browse repository at this point
Copy the full SHA 59ece5fView commit details
Commits on Apr 6, 2017
-
Configuration menu - View commit details
-
Copy full SHA for 3b0d371 - Browse repository at this point
Copy the full SHA 3b0d371View commit details -
Configuration menu - View commit details
-
Copy full SHA for bca8404 - Browse repository at this point
Copy the full SHA bca8404View commit details -
Configuration menu - View commit details
-
Copy full SHA for b0fb518 - Browse repository at this point
Copy the full SHA b0fb518View commit details -
Merge branch 'mac-updates' of http://github.com/ahknight/mergerfs int…
…o mac-updates
Configuration menu - View commit details
-
Copy full SHA for a00dc9b - Browse repository at this point
Copy the full SHA a00dc9bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6a67a92 - Browse repository at this point
Copy the full SHA 6a67a92View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3b2fa95 - Browse repository at this point
Copy the full SHA 3b2fa95View commit details -
Clients should get the xattrs they actually asked for, after all.
Configuration menu - View commit details
-
Copy full SHA for 25fe57f - Browse repository at this point
Copy the full SHA 25fe57fView commit details -
Configuration menu - View commit details
-
Copy full SHA for d0fa9ef - Browse repository at this point
Copy the full SHA d0fa9efView commit details -
Configuration menu - View commit details
-
Copy full SHA for cf567f1 - Browse repository at this point
Copy the full SHA cf567f1View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.