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

Add support for FUSE-T on macos #2191

Merged
merged 1 commit into from
Jun 24, 2023
Merged

Add support for FUSE-T on macos #2191

merged 1 commit into from
Jun 24, 2023

Conversation

macos-fuse-t
Copy link
Contributor

Add support for user space fuse (FUSE-T) on macos.
FUSE-T doesn't require installation of kernel extensions (KEXT) as opposed to osxfuse.
Details of the project can be found here: https://github.com/macos-fuse-t/fuse-t
To install FUSE-T use brew:
brew tap macos-fuse-t/homebrew-cask
brew install fuse-t

Relevant Issue (if applicable)

Details

Change autoconf to discover and use FUSE-T if present, otherwise fallback to osxfuse

@gaul
Copy link
Member

gaul commented Jun 22, 2023

@macos-fuse-t Please investigate CI failures.

@ggtakec fuse-t is an alternate FUSE implementation based on the macOS NFS client. In addition to a newer and maintained code base, this may give us a better path forward in CI which cannot use kexts.

Signed-off-by: Alex Fishman <alex@fuse-t.org>
Signed-off-by: alex <alex@alex-NUC10.lan>
@macos-fuse-t
Copy link
Contributor Author

please re-run CI

@gaul
Copy link
Member

gaul commented Jun 22, 2023

leap has a newer cppcheck version with new warnings which we encountered with Ubuntu 23.04 as well. I looked into this but suppressing some false positives was not easy and I need to look more carefully.

@gaul gaul merged commit 45e7cd0 into s3fs-fuse:master Jun 24, 2023
15 of 16 checks passed
@gaul
Copy link
Member

gaul commented Jun 24, 2023

Thank you for your contribution @macos-fuse-t! Please update the README with newer Homebrew instructions if you package this.

@ggtakec
Copy link
Member

ggtakec commented Jun 25, 2023

@macos-fuse-t Thank you for the corresponding PR of fuse-t!
I will prepare and post a PR to restore macos testing to the CI process.
(However since an error is output, it will be necessary to investigate the cause)

@macos-fuse-t
Copy link
Contributor Author

@ggtakec Sure, I will investigate the failures if any

@ggtakec
Copy link
Member

ggtakec commented Jun 25, 2023

@macos-fuse-t Thanks for your kindness.
I created a draft PR for #2198.
It looks like some tests are still failing, so I'll try to debug these.

@gaul
Copy link
Member

gaul commented Jun 25, 2023

@macos-fuse-t Does FUSE-T support the FUSE3 API?

@macos-fuse-t
Copy link
Contributor Author

No, fuse3 APIs are not supported at this moment. Future support is planned though.

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.

None yet

3 participants