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 fuse3 #325

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Add fuse3 #325

wants to merge 23 commits into from

Conversation

jfantinhardesty
Copy link
Contributor

What type of Pull Request is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Describe your changes in brief

This change makes fuse3 the default version of fuse for linux. It uses zig to link to libfuse3 for all linux deployments. Many of the fuse3 changes come directly from upstream.

Checklist

  • Tested locally
  • Added new dependencies
  • Updated documentation
  • Added tests

Related Issues

  • Related Issue #
  • Closes #

Copy link
Contributor

@foodprocessor foodprocessor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding a build option for fuse2 on Linux would be nice, and bringing the improvements from libfuse2_handler.go into libfuse_handler.go might be good. But those can both be backlog items.

TSG.md Outdated Show resolved Hide resolved
component/libfuse/libfuse_handler.go Show resolved Hide resolved
@Dabnsky
Copy link
Contributor

Dabnsky commented Oct 4, 2024

This is a pretty big one and I think this is one worth doing a group overview on. I could benefit from it as well as learning more about fuse3 and how it's being added here. I'm curious why libfuse2 has a windows build and libfuse has linux for example

@foodprocessor foodprocessor removed the request for review from brayan-trejo October 31, 2024 17:52
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.

4 participants