-
Notifications
You must be signed in to change notification settings - Fork 400
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
[#6404] feat (gvfs-fuse): Add Readme document of gvfs-fuse #6405
base: main
Are you sure you want to change the base?
Conversation
By the way, where do you plan to place this page? |
clients/filesystem-fuse/README.md
Outdated
|
||
# Execute the mount command in the foreground | ||
# mount the fileset uri to the local directory /mnt/gvfs | ||
./gvfs-fuse mount /mnt/gvfs gvfs://fileset/test/c1/s1/fileset1 -c /etc/gvfs_fuse.toml -f |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there any other place to describe how to config gvfs_fuse.toml
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this document is primarily intended to guide developers on how to build, test, and run the application.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
without gvfs_fuse.toml
, how should developers run gvfs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add some descriptions and examples
I think placing it in a submodule is sufficient. It is mainly for helping developers with building and testing. |
What changes were proposed in this pull request?
Add Readme document of gvfs-fuse
Why are the changes needed?
Fix: #6404
Does this PR introduce any user-facing change?
No
How was this patch tested?
No