-
Notifications
You must be signed in to change notification settings - Fork 265
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
Support freebsd #54
Support freebsd #54
Conversation
I really want to merge this, but haven't had time to look into it enough yet. Don't lose hope. |
fyi openbsd doesn't have ENODATA either. once this is merged i can work on openbsd support. |
friendly 'ping' :) |
ping, would love to have this merged in for the IPFS guys. |
Don't lose hope ;) |
Just tested this patch, it works. |
I pushed a cleaned up version of this as https://github.com/bazillion/fuse/tree/wip-freebsd 9cdcdc4. I'm not yet 100% convinced of all the details in the code, but more importantly: |
No progress on golang/go#9705 so far, and I'm afraid of merging this without working tests. Apart form that, looks good. If you want to make it happen faster, add Getxattr etc for freebsd to syscallx, on top of https://github.com/bazillion/fuse/tree/wip-freebsd bf5629a. |
Looks like golang/go#9705 finally got resolved by cl 6370. |
Current status: compiles but tests don't pass. FreeBSD FUSE seems to have way less features, need to skip a bunch of tests. |
Basic suppot for FreeBSD went in as 00c7ac3 . I'll be filing issues for the test failures, help appreciated. They'll show up under https://github.com/bazillion/fuse/issues?q=is%3Aopen+is%3Aissue+label%3Aplatform%3Afreebsd |
The only real ugliness here is ENODATA, which doesn't exist on freebsd.